Bug 970159 - Return integer 0 from sdp_attr_get_extmap_id(), not NULL pointer. r=jesup
authorChris Peterson <cpeterson@mozilla.com>
Sun, 09 Feb 2014 18:31:29 -0800
changeset 167816 063a9e3da435ec1edc27f42235feb26ac072d94b
parent 167815 afaec025122d7b1edc3be0d6a9d74eb34237c4d5
child 167821 14ceeedbb141c03ce6543cf1cdbc38aefb7057d8
child 167876 542f9093ec363842436c557f92552567e2b27174
push id26186
push usercbook@mozilla.com
push dateMon, 10 Feb 2014 11:48:02 +0000
treeherdermozilla-central@063a9e3da435 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjesup
bugs970159
milestone30.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 970159 - Return integer 0 from sdp_attr_get_extmap_id(), not NULL pointer. r=jesup
media/webrtc/signaling/src/sipcc/core/sdp/sdp_attr_access.c
--- a/media/webrtc/signaling/src/sipcc/core/sdp/sdp_attr_access.c
+++ b/media/webrtc/signaling/src/sipcc/core/sdp/sdp_attr_access.c
@@ -12529,17 +12529,17 @@ const char *sdp_attr_get_extmap_uri(void
  */
 u16 sdp_attr_get_extmap_id(void *sdp_ptr, u16 level,
                            u16 inst_num)
 {
     sdp_t       *sdp_p = (sdp_t *)sdp_ptr;
     sdp_attr_t  *attr_p;
 
     if (sdp_verify_sdp_ptr(sdp_p) == FALSE) {
-        return (NULL);
+        return 0;
     }
 
     attr_p = sdp_find_attr(sdp_p, level, 0, SDP_ATTR_EXTMAP, inst_num);
     if (attr_p == NULL) {
         if (sdp_p->debug_flag[SDP_DEBUG_ERRORS]) {
             CSFLogError(logTag, "%s extmap attribute, level %u instance %u "
                       "not found.", sdp_p->debug_str, level, inst_num);
         }