diff options
author | Zaheer Abbas Merali <zaheerabbas@merali.org> | 2008-02-27 23:29:44 +0000 |
---|---|---|
committer | Zaheer Abbas Merali <zaheerabbas@merali.org> | 2008-02-27 23:29:44 +0000 |
commit | 3bba6b5f9476c9e07f97ea236b55fd195d3d504c (patch) | |
tree | 77cdee915685da1cfa8348aacb655cbe421a180d | |
parent | 1507ada865aa5533aaf41df6f91357709004be01 (diff) | |
download | gst-plugins-bad-3bba6b5f9476c9e07f97ea236b55fd195d3d504c.tar.gz gst-plugins-bad-3bba6b5f9476c9e07f97ea236b55fd195d3d504c.tar.bz2 gst-plugins-bad-3bba6b5f9476c9e07f97ea236b55fd195d3d504c.zip |
sys/dvb/camutils.c: Remove the G_VALUE_HOLDS_BOXED checks. They were only added to help debug the data corruption.
Original commit message from CVS:
* sys/dvb/camutils.c:
Remove the G_VALUE_HOLDS_BOXED checks. They were
only added to help debug the data corruption.
-rw-r--r-- | ChangeLog | 8 | ||||
-rw-r--r-- | sys/dvb/camutils.c | 29 |
2 files changed, 19 insertions, 18 deletions
@@ -1,8 +1,14 @@ 2008-02-27 Zaheer Abbas Merali <zaheerabbas at merali dot org> * sys/dvb/camutils.c: + Remove the G_VALUE_HOLDS_BOXED checks. They were + only added to help debug the data corruption. + +2008-02-27 Zaheer Abbas Merali <zaheerabbas at merali dot org> + + * sys/dvb/camutils.c: Don't free the program descriptors, this structure - containing them is stills tored after. + containing them is still stored after. Fixes data corruption. 2008-02-27 Zaheer Abbas Merali <zaheerabbas at merali dot org> diff --git a/sys/dvb/camutils.c b/sys/dvb/camutils.c index 7e70aa26..b3062b59 100644 --- a/sys/dvb/camutils.c +++ b/sys/dvb/camutils.c @@ -232,14 +232,12 @@ cam_build_ca_pmt (GstStructure * pmt, guint8 list_management, guint8 cmd_id, streams = gst_structure_get_value (pmt, "streams"); value = gst_structure_get_value (pmt, "descriptors"); if (value != NULL) { - if (G_VALUE_HOLDS_BOXED (value)) { - program_descriptors = g_value_get_boxed (value); - /* get the length of program level CA_descriptor()s */ - len = get_ca_descriptors_length (program_descriptors); - if (len > 0) - /* add one byte for the program level cmd_id */ - len += 1; - } + program_descriptors = g_value_get_boxed (value); + /* get the length of program level CA_descriptor()s */ + len = get_ca_descriptors_length (program_descriptors); + if (len > 0) + /* add one byte for the program level cmd_id */ + len += 1; } lengths = g_list_append (lengths, GINT_TO_POINTER (len)); body_size += 6 + len; @@ -252,14 +250,12 @@ cam_build_ca_pmt (GstStructure * pmt, guint8 list_management, guint8 cmd_id, value = gst_structure_get_value (stream, "descriptors"); if (value != NULL) { - if (G_VALUE_HOLDS_BOXED (value)) { - stream_descriptors = g_value_get_boxed (value); - - len = get_ca_descriptors_length (stream_descriptors); - if (len > 0) - /* one byte for the stream level cmd_id */ - len += 1; - } + stream_descriptors = g_value_get_boxed (value); + + len = get_ca_descriptors_length (stream_descriptors); + if (len > 0) + /* one byte for the stream level cmd_id */ + len += 1; } lengths = g_list_append (lengths, GINT_TO_POINTER (len)); @@ -267,7 +263,6 @@ cam_build_ca_pmt (GstStructure * pmt, guint8 list_management, guint8 cmd_id, } } - buffer = g_malloc0 (body_size); body = buffer; |