[Bug 750325] rtcpbuffer: Update package validation to support reduced size rtcp packets

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Jun 3 09:19:47 PDT 2015


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

Olivier CrĂȘte <olivier.crete at ocrete.ca> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #304487|none                        |needs-work
             status|                            |

--- Comment #10 from Olivier CrĂȘte <olivier.crete at ocrete.ca> ---
Review of attachment 304487:
 --> (https://bugzilla.gnome.org/review?bug=750325&attachment=304487)

::: gst-libs/gst/rtp/gstrtcpbuffer.h
@@ +229,3 @@

+gboolean        gst_rtcp_buffer_validate_data_full      (guint8 *data, guint
len, gboolean allow_reduced_size_rctp);
+gboolean        gst_rtcp_buffer_validate_full           (GstBuffer *buffer,
gboolean allow_reduced_size_rctp);

Can't you just make it into a
gst_rtsp_buffer_validate_reduced() or something like that. Or if you want ot
make a _full(), make the argument be flags, although I'm not sure what other
flags they could be. Guessing what gst_rtcp_buffer_validate_full (buf, TRUE)
means is non easy without having devhelp opened. Something like
gst_rtsp_buffer_validate_full(buf, GST_RTCP_VALIDATE_ACCEPT_REDUCED) is much
more clear. Although my first option is to just make a function without
arguments or flags.

-- 
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