After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 703920 - h264: reference list count less than num_ref
h264: reference list count less than num_ref
Status: RESOLVED NOTABUG
Product: gstreamer-vaapi
Classification: Other
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gstreamer-vaapi maintainer(s)
gstreamer-vaapi maintainer(s)
Depends on:
Blocks: 720305
 
 
Reported: 2013-07-10 06:24 UTC by congx.zhong
Modified: 2014-04-26 12:33 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
h264-fix-reference-list-count-less-than-num_ref (1.75 KB, patch)
2013-07-10 06:24 UTC, congx.zhong 		rejected Details | Review

Description congx.zhong 2013-07-10 06:24:16 UTC 
Created attachment 248801 [details] [review]
h264-fix-reference-list-count-less-than-num_ref

Sometimes RefPicListx_count is less than the num_ref_listx because there's not enough reference pictures in dpb.
Comment 1 Gwenole Beauchesne 2013-07-12 13:37:34 UTC 
Hi,

(In reply to comment #0)
> Created an attachment (id=248801) [details]
> h264-fix-reference-list-count-less-than-num_ref
> 
> Sometimes RefPicListx_count is less than the num_ref_listx because there's not
> enough reference pictures in dpb.

Would you have a sample that exhibits the problem fixed by that patch? That's because if RefPicListX_count < num_ref_listX, then other slots are filled in with NULL and the various marking process functions normally do check for NULL pictures. So, I would like to understand what this fixes. Thanks.
Comment 2 Gwenole Beauchesne 2013-12-12 07:30:44 UTC 
Hi, any more recent updates on this one? Thanks.
Comment 3 Gwenole Beauchesne 2014-04-26 11:09:04 UTC 
Review of attachment 248801 [details] [review]:

This patch doesn't look goot to me. Besides, it doesn't seem to fix anything after testing, and only bring regressions instead:
CAWP5_TOSHIBA_E
CANL1_TOSHIBA_G
BA1_FT_C
Comment 4 Gwenole Beauchesne 2014-04-26 12:33:48 UTC 
Closing as per comment #1