Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@tsearle commented on this pull request.
In src/modules/rtpengine/rtpengine.c:
> @@ -4545,9 +4546,21 @@ static void parse_call_stats_1(struct minmax_mos_label_stats *mmls,
(char *)stream->child->iov[1].iov_base);
LM_DBG("rtpengine: XXX stream child val type %i\n",
stream->child->sibling->type);
- if((ssrc = bencode_dictionary_get_integer(stream, "SSRC", -1))
- == -1)
- continue;
+ ssrc = bencode_dictionary_get_integer(stream, "SSRC", -1);
+ if(ssrc == -1) {
+ ingress_ssrcs = bencode_dictionary_get_expect(
+ stream, "ingress SSRCs", BENCODE_LIST);
+ if(!ingress_ssrcs || !ingress_ssrcs->child)
+ continue;
+ LM_DBG("rtpengine: XXX got ingress SSRCs\n");
I'm not sure I fully understand the question... but this is here intentionally to make sure that ingress_ssrcs exists and has at least 1 element.
and the log message to indicate the affirmative
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.