[Bug 721715] h264parse: Multiple SEI messages in SEI RBSP

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=721715
  GStreamer | gst-plugins-bad | 1.2.2

Aurélien Zanelli <aurelien.zanelli> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #265549|0                           |1
        is obsolete|                            |

--- Comment #3 from Aurélien Zanelli <aurelien.zanelli at parrot.com> 2014-01-08 13:06:15 UTC ---
Created an attachment (id=265701)
 View: https://bugzilla.gnome.org/attachment.cgi?id=265701
 Review: https://bugzilla.gnome.org/review?bug=721715&attachment=265701

h264parse: Fix multiple SEI messages in one SEI RBSP.

I agree for the use of GArray instead of GList, it is more simple. Patch has
been updated.

Also i have a parsing bug which could be caused by more_rbsp_data()
implementation as described in bug 685890:
https://bugzilla.gnome.org/show_bug.cgi?id=685890
For example a SEI NAL payload (extracted from ts file i provided) is:
"00 01 C0 01 01 14 06 01 C4 80 00"
It contains 3 messages: buffering period (00 01 C0), picture timing (01 01 14)
and recovery point (06 01 C4). After the third message, I think
more_rbsp_data() should return FALSE, but actually it returns TRUE because
there is more than 8 bits remaining.
I will investigate this issue.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.