Remove useless string copy, and make sscanf safe again
authorRussell Bryant <russell@russellbryant.com>
Thu, 12 Feb 2009 21:41:01 +0000 (21:41 +0000)
committerRussell Bryant <russell@russellbryant.com>
Thu, 12 Feb 2009 21:41:01 +0000 (21:41 +0000)
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@175368 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_sip.c

index b5dbdec..f0a7fb9 100644 (file)
@@ -7671,7 +7671,6 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action
        iterator = req->sdp_start;
        while ((a = get_sdp_iterate(&iterator, req, "a"))[0] != '\0') {
                char mimeSubtype[128];
-               ast_copy_string(mimeSubtype, a, sizeof(mimeSubtype));
                if (option_debug > 1) {
                        int breakout = FALSE;
                
@@ -7759,7 +7758,7 @@ static int process_sdp(struct sip_pvt *p, struct sip_request *req, int t38action
                        }
                        red_cp = red_fmtp;
 
-               } else if (sscanf(a, "rtpmap: %u %[^/]/", &codec, mimeSubtype) == 2) {
+               } else if (sscanf(a, "rtpmap: %u %127[^/]/", &codec, mimeSubtype) == 2) {
                        /* We have a rtpmap to handle */
 
                        if (last_rtpmap_codec < SDP_MAX_RTPMAP_CODECS) {