summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVíctor Manuel Jáquez Leal <victorx.jaquez@intel.com>2015-09-11 16:49:16 +0200
committerVíctor Manuel Jáquez Leal <victorx.jaquez@intel.com>2015-09-15 19:36:37 +0200
commitf6ae00a6bb5d55b48fc2074b5abccad0bccee641 (patch)
tree47c1cad7397bfe40caea523b8f17bcccdc3064e5
parent6d0c2a8e23ee69292a9e107f0d60e378f6c623dd (diff)
downloadgst-vaapi-f6ae00a6bb5d55b48fc2074b5abccad0bccee641.tar.gz
decoder: h264: initialize PPS's slice_group_id
When the GstVaapiParserInfoH264 is allocated, the memory is not initialized, so it contains random data. When gst_h264_parser_parse_pps() fails, the PPS structure keeps slice_group_id pointer uninitialized, leading to a segmentation fault when the memory is freed. This patch prevents this by initializing the slice_group_id before the PPS parsing. Signed-off-by: Víctor Manuel Jáquez Leal <victorx.jaquez@intel.com> https://bugzilla.gnome.org/show_bug.cgi?id=754845
-rw-r--r--gst-libs/gst/vaapi/gstvaapidecoder_h264.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/gst-libs/gst/vaapi/gstvaapidecoder_h264.c b/gst-libs/gst/vaapi/gstvaapidecoder_h264.c
index 3947a445..55cb3962 100644
--- a/gst-libs/gst/vaapi/gstvaapidecoder_h264.c
+++ b/gst-libs/gst/vaapi/gstvaapidecoder_h264.c
@@ -1713,6 +1713,7 @@ parse_pps(GstVaapiDecoderH264 *decoder, GstVaapiDecoderUnit *unit)
standard but that should get a default value anyway */
pps->slice_group_map_type = 0;
pps->slice_group_change_rate_minus1 = 0;
+ pps->slice_group_id = NULL;
result = gst_h264_parser_parse_pps(priv->parser, &pi->nalu, pps);
if (result != GST_H264_PARSER_OK)