mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2024-11-28 12:41:05 +00:00
Code Issues 1 Projects Releases Wiki Activity
26655 commits 44 branches 1509 tags 436 MiB
Commit graph

189 commits

Author SHA1 Message Date
Jan Schmidt
f150cf2d2e Remove a bunch of silly ';;' typos at the end of lines 2015-03-12 01:33:03 +11:00
Arun Raghavan
70ed4fd438 mpegts: Fix a compiler warning 
Causes the following warning on clang:

gst-dvb-section.c:567:36: error: format specifies type 'unsigned long' but the argument has type 'int' [-Werror,-Wformat]
          descriptors_loop_length, end - 4 - data);
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
 2015-02-24 19:06:30 +05:30
Julien Isorce
857456b7ba mpegts: fix some compiler warnings 
i686-apple-darwin11-llvm-gcc-4.2 (GCC) 4.2.1

"warning: comparison is always true due to limited
range of data type"

https://bugzilla.gnome.org/show_bug.cgi?id=740674
 2014-11-25 10:23:43 +01:00
Ramprakash Jelari
f0e418b629 dvbsection: Fix format string compiler warning 2014-11-24 12:51:34 +01:00
Ramprakash Jelari
cbe46c0373 mpegtsdescriptor: Fix format string compiler warning 
gstmpegtsdescriptor.c:778:16: warning: format specifies type 'unsigned long' but the argument has type 'int' [-Wformat]
 2014-11-24 12:32:37 +01:00
Luis de Bethencourt
f76d1a2514 mpegts: remove storage of never used values 
Both _parse_atsc_mgt() and  _parse_atsc_vct () change the value of the variable
data just before returning. The new value is never used since data is a pointer
declared at the beginning of the function and going out of scope just after the
new value is stored.

https://bugzilla.gnome.org/show_bug.cgi?id=739404
 2014-10-30 23:19:59 +00:00
Felix Schwarz
a2b82ba541 mpegts: fix typos in comments 
https://bugzilla.gnome.org/show_bug.cgi?id=738072
 2014-10-07 13:11:13 +01:00
Reynaldo H. Verdejo Pinochet
8059cf6567 mpegts: fix typo in GstMpegtsExtendedEventDescriptor doc 2014-10-03 23:27:03 -03:00
Edward Hervey
9e73b487c4 mpegts: Add parsing for CA descriptor (0x05) 
https://bugzilla.gnome.org/show_bug.cgi?id=732986
 2014-07-11 08:32:06 +02:00
Sebastian Rasmussen
fba6ebaae2 mpegts: No need to check for NULL before calling g_free() 
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=732789
 2014-07-09 12:04:17 +02:00
Sebastian Rasmussen
66c38b50f3 mpegts: Don't confuse slice allocator with regular one 
Previously selector_bytes and private_data_bytes were sometimes allocated and
free using the normal allocator and sometimes using the slice allocator.
Additionally prefer g_strdup() to g_memdup() for strings.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=732789
 2014-07-09 12:04:17 +02:00
Edward Hervey
7626b494cb mpegts: Fix dvb linkage leaks 
CID #1224130
 2014-07-09 07:50:13 +02:00
Stefan Ringel
43dd0ec62a mpegts: use getter for egde linkage descriptor type 
https://bugzilla.gnome.org/show_bug.cgi?id=730914
 2014-07-09 07:41:33 +02:00
Stefan Ringel
41f1c8c32e mpegts: fix annotation 
https://bugzilla.gnome.org/show_bug.cgi?id=730914
 2014-07-09 07:41:32 +02:00
Edward Hervey
90b27a3c8b mpegts: Add padding to public structures 
Allows use to add API in the future without breaking ABI. We broke the API/ABI
once between 1.2 and 1.4, let's try to avoid this in the future even if this
is an unstable library.

https://bugzilla.gnome.org/show_bug.cgi?id=730914
 2014-07-09 07:39:37 +02:00
Thiago Santos
1d233b7516 mpegts: atsc: fix leak of short name 2014-07-08 15:43:32 -03:00
Thiago Santos
2874bff392 mpegts: fix leak of language code 2014-07-08 14:56:38 -03:00
Edward Hervey
8c53dfcfb6 mpegts: Expose GstMpegtsDescriptor free function 
Nothing earth shattering, but avoids people having to use g_boxed_free()
 2014-07-01 10:03:49 +02:00
Sebastian Dröge
426f9e6097 mpegts: Fix memory leaks in the test and the library 
Currently uses g_boxed_free() in the unit test, this needs to
be fixed properly when cleaning up the API.

https://bugzilla.gnome.org/show_bug.cgi?id=730914
 2014-06-29 21:34:50 +02:00
Thiago Santos
e6244874ab mpegts: atsc: avoid calling g_convert with 0-sized input 
Avoids an assertion. The cached string will be assigned an empty string
as its value when size is 0.
 2014-06-27 12:50:27 -03:00
Edward Hervey
22dfd9aef3 mpegts: Unify API : GstMpegts / gst_mpegts_ / GST_*_MPEGTS_ 
It was previously a mix and match of both variants, introducing just too much
confusion.

The prefix are from now on:
* GstMpegts for structures and type names (and not GstMpegTs)
* gst_mpegts_ for functions (and not gst_mpeg_ts_)
* GST_MPEGTS_ for enums/flags (and not GST_MPEG_TS_)
* GST_TYPE_MPEGTS_ for types (and not GST_TYPE_MPEG_TS_)

The rationale for chosing that is:
* the namespace is shorter/direct (it's mpegts, not mpeg_ts nor mpeg-ts)
* the namespace is one word under Gst
* it's shorter (yah)
 2014-06-25 14:50:05 +02:00
Edward Hervey
cddabd751c mpegtssection: Free data in error cases 
Data is marked as "transfer full", if we have an error we need to
free it.
 2014-06-25 14:15:37 +02:00
Edward Hervey
42b3d6ec8a mpegts: More doc fixes 
Still not done :(
 2014-06-24 16:06:01 +02:00
Edward Hervey
0607756e24 mpegts: more annotation fixes 2014-06-24 16:06:01 +02:00
Stefan Ringel
7c884374eb mpegts: change to GType and from gchar[] to gchar* 
https://bugzilla.gnome.org/show_bug.cgi?id=730914
 2014-06-24 16:06:01 +02:00
Edward Hervey
123f642d5e mpegts: Fix descriptor_from_dvb_service 
Various leaks/overwrites issues
 2014-06-24 14:51:32 +02:00
Edward Hervey
aaee4dc23b all: Don't declare variables in 'for' loops 2014-06-24 14:51:32 +02:00
Edward Hervey
a57005abbc mpegts: Remove unneeded NULL check 
Allocating those structures is done in all code paths leading there

CID #1219860
CID #1219861
CID #1219862
CID #1219863
CID #1219864
 2014-06-02 08:24:08 +02:00
Edward Hervey
86b8388c3f atsc-section: Fix annotation 2014-05-29 14:29:16 +02:00
Edward Hervey
611edd5e96 dvb-descriptor: Fix multilingual service name parsing 
the provider and service name was inverted
 2014-05-29 14:29:15 +02:00
Stefan Ringel
6a622a678e example: ts-parser: add parential rating descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=730854
 2014-05-29 14:29:15 +02:00
Stefan Ringel
c35dc33b7f example: ts-parser: add content descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=730854
 2014-05-29 14:29:15 +02:00
Stefan Ringel
1c9bbb1e00 example: ts-parser: add component descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=730854
 2014-05-29 14:29:15 +02:00
Stefan Ringel
f6fdd83b7e mpegts: bugfix mobile hand over linkage parse missmatch 
https://bugzilla.gnome.org/show_bug.cgi?id=730901
 2014-05-29 11:26:21 +02:00
Thiago Santos
fa1eb09520 mpegts: atsc: add missing field to ETT table 
Set the subtable_extension as ett_table_id_extension
for ETT tables as it is used by it

https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:38:00 +02:00
Thiago Santos
1cab60b713 mpegts: atsc: add atsc's EIT table parsing 
ATSC has its own version of the EIT table (DVB also has one).

This patch adds parsing for the ATSC EIT table and also fixed
the section identification to mark it as the ATSC one.

The implementation aws refactored to reuse some common internal
structures from ETT.

Also adds its dumping function to ts-parser example

https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:38:00 +02:00
Thiago Santos
0c6bb561e0 mpegts: atsc: add STT table parsing 
Adds the system time table structure and functions for convenient parsing of
it and for getting the UTC datetime that it represents. Also adds its
information dumping to the ts-parser example

https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:37:59 +02:00
Thiago Santos
0c0379e8f0 mpegts: atsc: add encoding conversion for UTF-16 ETT strings 
https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:37:59 +02:00
Thiago Santos
7ced36eccd mpegts: atsc: add ETT structures and parsing 
ETT (extended text table) contains ATSC text information with descriptions
of virtual channels and events. The text can be internationalized and also
compressed.

https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:37:59 +02:00
Thiago Santos
9eed28d353 mpegts: add atsc MGT section parsing 
Add a parsing function for MGT and also detect the EIT tables
for ATSC, the EIT pids are reported inside the MGT and we are still
only relying only on the table id for detecting it. In the future we
would want to also check the pid and compare with whatever the MGT
previously reported to confirm that it is indeed the EIT.

https://bugzilla.gnome.org/show_bug.cgi?id=730435
 2014-05-29 10:37:59 +02:00
Thiago Santos
8f6a3c0a99 mpegts: atsc: rename TVCT to VCT as it is the same as CVCT 
Make the ATSC section parse handle both TVCT and CVCT as they are
nearly the same struct (CVCT uses 2 reserved bits that are ignored
in TVCT).

This is changing the glib type and the struct name but TVCT wasn't
released in a stable package yet so there should be no problem.

Also includes some parsing fixes and changes short_name to be
directly stored as utf8 rather than utf16

https://bugzilla.gnome.org/show_bug.cgi?id=730642
 2014-05-29 10:37:59 +02:00
Edward Hervey
6e1679b9cb mpegts: Make some fields optional 
Some people might just not care about the contents
 2014-05-28 15:55:40 +02:00
Edward Hervey
ca2847e234 mpegts: Update docs for DVB frequency list 2014-05-28 15:55:40 +02:00
Edward Hervey
7852430934 mpegts: Fix Logical Channel Descriptor parsing 
No wonder we were getting garbage ...
 2014-05-28 08:03:09 +02:00
Edward Hervey
1797802bd2 mpegts: Ensure NULL-terminated language code 
Yup, it breaks API and ABI, I know. But let's avoid people and bindings
printing out garbage
 2014-05-27 17:34:48 +02:00
Edward Hervey
f7b1b5fcd3 mpegts: Fix Extended DVB descriptor parsing 
We were over-reading data, and the nb_items field isn't needed,
it's a property of the GPtrArray
 2014-05-27 17:34:48 +02:00
Edward Hervey
b207a15073 mpegts: Fix usage of memcpy 
source and destination were inverted
 2014-05-27 16:17:39 +02:00
Stefan Ringel
0168160863 mpegts: add multilingual component descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728429
 2014-05-27 15:34:40 +02:00
Stefan Ringel
b0f3892ae5 mpegts: add multilingual service name descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728429
 2014-05-27 15:34:40 +02:00
Stefan Ringel
cac329a621 mpegts: add multilingual bouquet name descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728429
 2014-05-27 15:34:39 +02:00
Stefan Ringel
b4cc5a2177 mpegts: add multilingual network name descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728429
 2014-05-27 15:34:39 +02:00
Edward Hervey
42c061b0e7 mpegts: Fix Bouquet Name parsing 
the field is not length prefixed
 2014-05-26 11:54:50 +02:00
Stefan Ringel
bc7cf1520c mpegts: add bouquet name descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728364
 2014-05-26 11:18:20 +02:00
Stefan Ringel
985d19deaa mpegts: add stuffing descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728364
 2014-05-26 11:18:20 +02:00
Stefan Ringel
570f78df79 mpegts: add service list descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=728364
 2014-05-26 11:18:19 +02:00
Sebastian Dröge
752b3a503e dvb-section: Don't compare unsigned integers for >= 0 
gst-dvb-section.c:93:12: error: comparison of unsigned expression >= 0 is always true
      [-Werror,-Wtautological-compare]
  if (hour >= 0 && hour < 24 && minute >= 0 && minute < 60 && second >= 0
      ~~~~ ^  ~
gst-dvb-section.c:93:40: error: comparison of unsigned expression >= 0 is always true
      [-Werror,-Wtautological-compare]
  if (hour >= 0 && hour < 24 && minute >= 0 && minute < 60 && second >= 0
                                ~~~~~~ ^  ~
gst-dvb-section.c:93:70: error: comparison of unsigned expression >= 0 is always true
      [-Werror,-Wtautological-compare]
  if (hour >= 0 && hour < 24 && minute >= 0 && minute < 60 && second >= 0
                                                              ~~~~~~ ^  ~
 2014-05-21 08:53:03 +02:00
Thiago Santos
0cdcc5a7b1 mpegts: Check hour:minute:second values before creating datetime 
Avoids assertions when the packets are malformed and also add the case
where the time isn't specified, only the date.
 2014-05-19 14:55:21 -03:00
Thiago Santos
43f3f837fa mpegts: mpegtsdescriptor: fix gobject-introspection warnings 
Adds missing : to gobject-introspection markers
 2014-05-15 21:34:31 -03:00
Thiago Santos
f99fe4747e mpegts: sections: prevent assertion when packet parsing fails 
the descriptors entry can be left as NULL and freeing the structure
will fail (assertion happens)
 2014-05-15 19:25:06 -03:00
Edward Hervey
2b92709154 mpegts: Fix leak 
The cell extension was never added to the parent cell

CID #1212162
 2014-05-09 14:32:49 +02:00
Stefan Ringel
eeaf6e4811 mpegts: add dvb ca identifier descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727560
 2014-04-16 09:23:23 +02:00
Stefan Ringel
0c773b8cdd mpegts: add scrambling descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727560
 2014-04-16 09:23:05 +02:00
Stefan Ringel
e535967ee9 mpegts: add frequency list descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727560
 2014-04-16 09:22:46 +02:00
Stefan Ringel
ace60abef5 mpegts: add data broadcast id descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727403
 2014-04-16 09:21:45 +02:00
Stefan Ringel
065abf6d54 mpegts: add private data specifier descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727403
 2014-04-16 09:21:45 +02:00
Sebastian Dröge
8407ee9ee9 dvb: Minor code style fix 2014-04-16 09:21:45 +02:00
Stefan Ringel
e71c264dbb mpegts: add parential rating descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727403
 2014-04-16 09:21:45 +02:00
Stefan Ringel
eb246e2bfd mpegts: add linkage descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=727403
 2014-04-16 09:21:44 +02:00
Tim-Philipp Müller
990efe1fc0 mpegts: for loop initial declarations are only allowed in C99 mode 
Fixes compiler warnings.
 2014-04-16 00:28:37 +01:00
Stefan Ringel
f566d17202 mpegts: add atsc terrestrial virtual channel table 
https://bugzilla.gnome.org/show_bug.cgi?id=727460
 2014-04-10 09:09:38 +02:00
Vincent Penquerc'h
8f4123a881 mpegts: fix sdt length check 
It was originally test for 9 bytes (as the comment says) and was
rewritten buggily. So rewrite it a third way, which is now
hopefully consistent with the original and the comment, while
being more sense-making to humans.

Coverity 1139654
 2014-04-09 16:01:43 +01:00
Vincent Penquerc'h
195470a785 mpegts: turns out glib aborts on allocation failure 
So this is actually pointless. We'll just have to ignore
Coverity moaning on those.

Revert "mpegts: test for allocation failure"

This reverts commit 224cb81b8f.
 2014-04-09 16:01:43 +01:00
Vincent Penquerc'h
224cb81b8f mpegts: test for allocation failure 
While it's unlikely to get there, it silences the coverity warning
on the error code path that we test for NULL before freeing, when
all branches there are from locations where pmt cannot be NULL,
and removing the NULL check makes the code more vulnerable to a
hypothetical future branch from somewhere where it can be.

Coverity 1139852
 2014-04-09 15:44:10 +01:00
Stefan Ringel
73862d5096 mpegts: fix docs 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727356
 2014-03-31 15:05:48 +02:00
Stefan Ringel
b7efacfe15 mpegts: add dvb-t2 delivery system descriptor 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727356
 2014-03-31 15:05:48 +02:00
Stefan Ringel
e8c86e355f mpegts: bugfix delivery system descriptors 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=726979
 2014-03-31 15:02:01 +02:00
Stefan Ringel
d108fe7ee1 mpegts: use GST_READ_UINT32_BE 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727187
 2014-03-31 15:00:54 +02:00
Stefan Ringel
8b8ceb2dd9 mpegts: add parsing data broadcast descriptor 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727187
 2014-03-31 15:00:54 +02:00
Stefan Ringel
f8b7a130cc mpegts: add parsing stream identifier 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727187
 2014-03-31 15:00:54 +02:00
Stefan Ringel
2e051cf412 mpegts: use glib alloc and free 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=727159
 2014-03-31 14:54:17 +02:00
Stefan Ringel
fd8bda1c03 dvbsrc: mpegts: change bandwidth to DVBv5 values 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=725463
 2014-03-15 18:33:18 +01:00
Stefan Ringel
40e7bfd35d docs: mpegts: add missed docs 
Signed-off-by: Stefan Ringel <linuxtv@stefanringel.de>

https://bugzilla.gnome.org/show_bug.cgi?id=725278
 2014-02-27 08:48:17 +01:00
Jesper Larsen
a9b1fdd948 mpegts: Add constructor for dvb service descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=724981
 2014-02-25 14:34:29 +01:00
Jesper Larsen
d43e7173d8 mpegts: Add support for creation of SDT 
https://bugzilla.gnome.org/show_bug.cgi?id=724981
 2014-02-25 14:34:29 +01:00
Jesper Larsen
3fe1608e83 mpegts: Add transport_stream_id to GstMpegTsSDT 
The transport_stream_id is contained in the subtable extension.
Introduce a variable for readability.

https://bugzilla.gnome.org/show_bug.cgi?id=724981
 2014-02-25 14:34:29 +01:00
Edward Hervey
63145f4576 mpegts: Fix descriptor checks 
Only use g_return_val_if_fail on provided direct arguments. The others get
checked all the time.

https://bugzilla.gnome.org/show_bug.cgi?id=724464
 2014-02-25 10:19:23 +01:00
Jesper Larsen
e28b6fd523 mpegts: network_name: Check converted length 
Check the length of the name argument after it has been converted.

https://bugzilla.gnome.org/show_bug.cgi?id=723953
 2014-02-24 16:01:11 +01:00
Brendan Long
cd3c3ebf01 mpegts: Add gst_mpegts_section_get_data, to allow us to get the original binary section data. 
https://bugzilla.gnome.org/show_bug.cgi?id=724255
 2014-02-20 19:19:03 +01:00
Edward Hervey
142233d917 mpegts: Add size guards for descriptors where neeeded 
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=724464
 2014-02-20 19:08:33 +01:00
Edward Hervey
4190a1717a mpeg-ts: Rename private function 
Calling it _desc_checks when it was in fact used for sections was a
tad confusing ...
 2014-02-20 18:14:51 +01:00
Stefan Ringel
beb31d127b mpegts: Add support for content descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=724034
 2014-02-17 08:55:37 +01:00
Stefan Ringel
792fc1cf01 mpegts: Add support for component descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=724034
 2014-02-17 08:55:12 +01:00
Stefan Ringel
347ac5fbd7 mpegts: Add support for extended event desriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=724034
 2014-02-17 08:54:37 +01:00
Stefan Ringel
e6cb996cea mpegts: Add support for DVB terrestrial delivery system descriptor 
https://bugzilla.gnome.org/show_bug.cgi?id=724069
 2014-02-16 10:13:53 +01:00
Jesper Larsen
603714a405 mpegts: pmt: Detect descriptor length correct 2014-02-08 18:41:06 +01:00
Jesper Larsen
f1c8de4bc2 mpegts: Correctly check for PAT,PMT,CAT,TSDT 2014-02-08 18:38:55 +01:00
Jesper Larsen
972f7ea71f mpegts: Fix some packetizing bugs 
- Length of NIT stream descriptors was not detected correct
- Reserved bits was not set according to EN 300 468, ISO/IEC 13818-1
- Also set output data size if the section was previously packetized

https://bugzilla.gnome.org/show_bug.cgi?id=723892
 2014-02-08 16:59:47 +01:00
Edward Hervey
042c8924b6 dvb-descriptor: Remove unneeded increment 2014-02-07 11:53:38 +01:00
Edward Hervey
63348811d2 dvb-descriptor: Fix comparision 
we want to stop if one OR the other is NULL.
 2014-02-07 11:53:28 +01:00
Jesper Larsen
dc968163da mpegts: Updated docs with new API 2014-02-06 16:01:35 +01:00