[Bug 732266] vaapidecode: h264: add support for decoding SVC base layers only

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Aug 28 23:27:34 UTC 2017


https://bugzilla.gnome.org/show_bug.cgi?id=732266

--- Comment #68 from sreerenj <bsreerenj at gmail.com> ---
(In reply to Orestis Floros from comment #67)
> By the way, isn't the code in gstvaapidecoder_h264.c:4589
> 
> > if (priv->is_avcC)
> >   result = gst_h264_parser_identify_nalu_avc (priv->parser,
> >       buf, 0, buf_size, priv->nal_length_size, &pi->nalu);
> > else
> >   result = gst_h264_parser_identify_nalu_unchecked (priv->parser,
> >       buf, 0, buf_size, &pi->nalu);
> > status = get_status (result);
> 
> dangerous in the same way? If h264parser fails to parse them and they have
> garbage data (eg as with the PPS case you mention) it can lead to a segfault
> in _finalize() which is called after goto exit IIUC.
> 

There is nothing to do with what h264pasr has been parsed or not.
But yes, there is a possibility to crash in this scenario since we are not zero
initializing the parser_info. Easy fix for this is to use
gst_vaapi_mini_object_new0() instead of gst_vaapi_mini_object_new() for
GstVaapiParserInfoH264 creation.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list