Comment # 3
on bug 12788
from Jaap Keuter
in packet-sdp.c:setup_sdp_transport() there's a note stating:
/* Increase the count of media channels, but don't walk off the end of
the arrays. */
/* XXX: I don't know why this was done here - I'm keeping it here in
case
* removing it causes problems, but it's wrong.
transport_info->media_count
* is already incremented in the while() loop above. Incrementing it
* again here will cause bugs.
And indeed seems. Removing these lines
if (transport_info->media_count < (SDP_MAX_RTP_CHANNELS-1))
transport_info->media_count++;
seems to solve this problem.
There's still crap in transport_info contents though, so there may be more.
And the note author could be right that this change causes other problems to
appear.
Needs further investigation.
You are receiving this mail because:
- You are watching all bug changes.
