[Bug 758384] dashdemux: tests: refactor into adaptive_engine components

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Nov 24 10:15:18 PST 2015


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

--- Comment #14 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> ---
Review of attachment 315947:
 --> (https://bugzilla.gnome.org/review?bug=758384&attachment=315947)

::: tests/check/elements/hls_demux.c
@@ +63,3 @@
+  }
+  memset (mpeg_ts->data, 0, length);
+  for (pos = 0; pos < length; pos += 188) {

This seems a bit dangerous since it only stays within bounds if length%188==0

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