diff options
| author | Pascal Quantin <pascal.quantin@gmail.com> | 2018-05-03 14:20:26 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2018-05-03 13:39:40 +0000 |
| commit | 47bf21a6c8dc11b91009cbb1a2b5ec723b1a9ea9 (patch) | |
| tree | 60ab6222bbba4842f213fd233986d5c8007109eb /epan | |
| parent | faf5f86e7e7616149ad5ce82c8f7662b89288064 (diff) | |
SDP: do not assume that media_desc pointer is always set
Bug: 14648
Change-Id: I9d585a1b49e15529b19414be52c8bf198475fd62
Reviewed-on: https://code.wireshark.org/review/27304
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-sdp.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/epan/dissectors/packet-sdp.c b/epan/dissectors/packet-sdp.c index 7f9eec6ed5..b92df903a0 100644 --- a/epan/dissectors/packet-sdp.c +++ b/epan/dissectors/packet-sdp.c @@ -2036,11 +2036,15 @@ static void dissect_sdp_media_attribute(tvbuff_t *tvb, packet_info *pinfo, proto tvb, offset, -1, ENC_UTF_8|ENC_NA); break; case SDP_RTCP : - if (!ws_strtou16(attribute_value, NULL, &media_desc->control_port)) - media_desc->control_port = 0; /* Just use default, if not legal port */ + if (media_desc) { + if (!ws_strtou16(attribute_value, NULL, &media_desc->control_port)) + media_desc->control_port = 0; /* Just use default, if not legal port */ + } break; case SDP_RTCP_MUX : - media_desc->control_port = media_desc->media_port; + if (media_desc) { + media_desc->control_port = media_desc->media_port; + } break; default: /* No special treatment for values of this attribute type, just add as one item. */ |