File 0001-aacparse-Make-sure-we-have-enough-data-in-... of Package gstreamer-plugins-good

Overview Repositories Revisions Requests Users Attributes Meta

File 0001-aacparse-Make-sure-we-have-enough-data-in-the-codec_data-to-be-able-to-parse-it.patch of Package gstreamer-plugins-good (Revision 85212ecff0b2243c2aef4eb1d164c29e)

Currently displaying revision 85212ecff0b2243c2aef4eb1d164c29e , Show latest

From 87a2c140ca54c5128093377e9b25a5c24b346727 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
Date: Thu, 1 Dec 2016 13:38:16 +0200
Subject: [PATCH] aacparse: Make sure we have enough data in the codec_data to
 be able to parse it

Also error out cleanly if mapping the buffer failed.

https://bugzilla.gnome.org/show_bug.cgi?id=775450
---
 gst/audioparsers/gstaacparse.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gst/audioparsers/gstaacparse.c b/gst/audioparsers/gstaacparse.c
index d9dc424..bf424b2 100644
--- a/gst/audioparsers/gstaacparse.c
+++ b/gst/audioparsers/gstaacparse.c
@@ -298,11 +298,12 @@ gst_aac_parse_sink_setcaps (GstBaseParse * parse, GstCaps * caps)
   if (value) {
     GstBuffer *buf = gst_value_get_buffer (value);
 
-    if (buf) {
+    if (buf && gst_buffer_get_size (buf) >= 2) {
       GstMapInfo map;
       guint sr_idx;
 
-      gst_buffer_map (buf, &map, GST_MAP_READ);
+      if (!gst_buffer_map (buf, &map, GST_MAP_READ))
+        return FALSE;
 
       sr_idx = ((map.data[0] & 0x07) << 1) | ((map.data[1] & 0x80) >> 7);
       aacparse->object_type = (map.data[0] & 0xf8) >> 3;

Places

File 0001-aacparse-Make-sure-we-have-enough-data-in-the-codec_data-to-be-able-to-parse-it.patch of Package gstreamer-plugins-good (Revision 85212ecff0b2243c2aef4eb1d164c29e)

Places