diff options
| author | Luis de Bethencourt <luis.bg@samsung.com> | 2015-01-08 15:36:04 +0000 |
|---|---|---|
| committer | Tim-Philipp Müller <tim@centricular.com> | 2016-02-16 00:24:40 +0000 |
| commit | e3d8d8cedba098c4c30df82eec67063c9bf2a08f (patch) | |
| tree | 8c64f0564d0f491d9611732d8ef5192057ae4565 | |
| parent | 59fea44503547fa731333235bae115c930375b69 (diff) | |
| download | gstreamer-plugins-good-e3d8d8cedba098c4c30df82eec67063c9bf2a08f.tar.gz | |
rtp: value truncated too short creates dead code
type is truncated to 0-31 with "& 0x1f", but right after that it is checks if
the value is equivalent to GST_H265_NAL_VPS, GST_H265_NAL_SPS, and
GST_H265_NAL_PPS (which are 32, 33, and 34 respectively). Obviously, this will
never be True if the value is maximum 31 after the truncation.
The intention of the code was to truncate to 0-63.
| -rw-r--r-- | gst/rtp/gstrtph265pay.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gst/rtp/gstrtph265pay.c b/gst/rtp/gstrtph265pay.c index 629c6e021..b2da44d99 100644 --- a/gst/rtp/gstrtph265pay.c +++ b/gst/rtp/gstrtph265pay.c @@ -821,7 +821,7 @@ gst_rtp_h265_pay_decode_nal (GstRtpH265Pay * payloader, GST_DEBUG ("NAL payload len=%u", size); header = data[0]; - type = header & 0x1f; + type = header & 0x3f; /* We record the timestamp of the last SPS/PPS so * that we can insert them at regular intervals and when needed. */ |