Patches item #1992667, was opened at 2008-06-13 08:38 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=637566&aid=1992667&group_id=104305
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: sipp Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: flefoll (flefoll) Assigned to: Nobody/Anonymous (nobody) Summary: get_remote_port_media() should accept 5-digit port numbers Initial Comment: call.cpp:get_remote_port_media() parses incoming message and looks for "m=audio " or "m=video " patterns, in order to read port number. Current code copies bytes including space after "audio" or "video" to a buffer limited to 6 bytes, including final null char. As a consequence, port 40000 will be understood as port 4000. Known effects : for instance, uac_pcap scenario won't be able to transmit RTP to a destination port above 9999. I first thought of just skipping initial space when copying to buffer, but I think that in fact, atoi() can just work on source message directly. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=637566&aid=1992667&group_id=104305 ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ Sipp-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/sipp-users
