Commit graph

1088 commits

Author SHA1 Message Date
Edward Hervey
f86751d9f7 asfdemux: Fix prerolling files with "empty" streams
This is a regression that was introduced by
commit 1803b3c185
"    asfdemux: Add support for dvr-ms"

The problem is that some files/streams might contain stream definition
but there is no actual packets for those streams.

This was used to "define" streams with different bitrates for example.

The first_ts calculation resulted in never ever finding a valid first_ts
since some streams were empty, and therefore never "activating" itself.

Instead of that we first check if we are prerolled. And if we are we
unconditionally get the "first_ts"

The preroll check has been adapted to check whether streams of
each defined type (i.e. audio/video/sub) has been prerolled. This solves
the problem of having different streams of a particular type where only
one stream actually has data.
2017-11-13 11:13:30 +01:00
Edward Hervey
2d9867b120 asfdemux: Store the stream type in streams
The field was present but never set :)
2017-11-13 11:09:31 +01:00
Satya Prakash Gupta
38b198255f realmedia: fix leak in asmrules error code path
https://bugzilla.gnome.org/show_bug.cgi?id=787056
2017-08-31 19:27:02 +01:00
Satya Prakash Gupta
c3cf7ab5c4 realmedia: Fix Memory leak in error case
https://bugzilla.gnome.org/show_bug.cgi?id=787014
2017-08-30 17:20:41 +02:00
Vivia Nikolaidou
3b99cf6c54 dvdlpmdec: Fix overrun from coverity
CID #1414801
2017-07-11 14:23:27 +03:00
Vivia Nikolaidou
cd59f39c2c dvdlpcmdec: Added support for Blu-Ray audio
https://bugzilla.gnome.org/show_bug.cgi?id=784552
2017-07-05 19:17:59 +03:00
Edward Hervey
ec24f4c9f2 asfdemux: More index handling
In the same vein as the previous commit, the end of the file might have been
cut and therefore contains bogus data.

If the object is too short, consider it as garbage and return a non-fatal
flow.

Regression introduced in 97294eb8bb
2017-06-27 11:49:21 +02:00
Edward Hervey
0bbdfa8ee9 asfdemux; Don't assume index is present at end of file
Some files have garbage at the end of the file, don't error out if we don't
find the index there.

Reverts back to previous behaviour (where corrupted/missing indexex were
not a fatal error).

Regression introduced in 97294eb8bb
2017-06-27 10:41:42 +02:00
Mark Nauwelaerts
3a25a3a948 asfdemux: adjust segment start to a keyframe ts only for a KEYUNIT seek
... which is how it has always been until changed coincidentally in commit
9ee60482b2 which then prompted commit
dba14178ce to consider ACCURATE as well
to compensate for that in a lateral way
2017-06-24 18:14:59 +02:00
Thiago Santos
42f8e22f17 asfdemux: fix checking for header objects
The if 'and' condition was mixed up with the last parameter of the
function and would lead to wrong results
2017-06-06 21:48:19 -07:00
Nicolas Dufresne
d02e7d8029 Remove plugin specific static build option
Static and dynamic plugins now have the same interface. The standard
--enable-static/--enable-shared toggle are sufficient.
2017-05-16 14:07:56 -04:00
Nicolas Dufresne
c1e526a879 Rename plugin filenames to match plugin names
libgstrmdemux.so becomes libgstrealmedia.so

https://bugzilla.gnome.org/show_bug.cgi?id=779344
2017-03-08 20:04:25 -05:00
Sebastian Dröge
66659f07f5 asfdemux: Fix compilation with gcc 7
gstasfdemux.c: In function 'gst_asf_demux_parse_stream_object':
gstasfdemux.c:3001:39: error: '<<' in boolean context, did you mean '<' ? [-Werror=int-in-bool-context]
   is_encrypted = ! !((flags & 0x8000) << 15);
                     ~~~~~~~~~~~~~~~~~~^~~~~~

https://bugzilla.gnome.org/show_bug.cgi?id=779329
2017-02-27 20:55:34 +02:00
Sebastian Dröge
d21017b52a asfdemux: Check that we have enough data available before parsing bool/uint extended content descriptors
https://bugzilla.gnome.org/show_bug.cgi?id=777955
2017-01-31 13:50:21 +02:00
Sebastian Dröge
dec880031d asfdemux: Reset number of languages to 0 when freeing the array because of errors
Otherwise we will happily index into the array at NULL if the requested
index is smaller than the number of languages that were previously
allocated.

https://bugzilla.gnome.org/show_bug.cgi?id=777937
2017-01-30 15:39:59 +02:00
Sebastian Dröge
0ff3f79330 asfdemux: Fix memory leak of payload extensions for unusable/unsupported streams
https://bugzilla.gnome.org/show_bug.cgi?id=775515
2016-12-02 14:06:21 +02:00
Sebastian Dröge
cddfdaab78 real: Fix compiler warnings with latest clang
realhash.c:295:11: error: implicit conversion from 'int' to 'char' changes value from 128 to -128 [-Werror,-Wconstant-conversion]
  *buf1 = 128;
        ~ ^~~

https://bugzilla.gnome.org/show_bug.cgi?id=775112
2016-11-25 23:51:23 +02:00
Edward Hervey
9ad9e06fcb asfdemux: Allow 64bit sizes for DATA object
we *can* have files greater than 4GB, any other object should be
smaller than that
2016-11-25 12:41:03 +01:00
Edward Hervey
4a52e6c827 asfdemux: Add sanity check for asf_stream_audio
We should have enough bytes for the specified codec_data

https://bugzilla.gnome.org/show_bug.cgi?id=775070
2016-11-25 10:08:31 +01:00
Edward Hervey
8f05e8f449 asfdemux: Add sanity checks when reading asf_stream_video_format
It should report a size of at least 40 bytes
Also check for the size of the remaining data (i.e. codec_data)

https://bugzilla.gnome.org/show_bug.cgi?id=775070
2016-11-25 10:07:21 +01:00
Edward Hervey
97294eb8bb asfdemux: Add check for invalid/corrupt asf object
An asf object can't realistically be bigger than 2**32 bytes.

If it reports a size bigger than that, consider it corrupt and properly
propagate the error back.

https://bugzilla.gnome.org/show_bug.cgi?id=775070
2016-11-25 09:46:12 +01:00
Edward Hervey
48a493070a asfdemux: Handle EOS in push-mode on corrupted files
It is possible no streams were activated when receiving EOS, if so
handled it as if we hadn't seen the header

https://bugzilla.gnome.org/show_bug.cgi?id=774846
2016-11-22 18:21:46 +01:00
Edward Hervey
e7ff86665a asfdemux: FILE headers are mandatory in the header
As per the specification, also avoids ending up trying to play a
file with plenty of un-initialized values.

https://bugzilla.gnome.org/show_bug.cgi?id=774846
2016-11-22 18:21:46 +01:00
Edward Hervey
cc04255e94 asfdemux: Handle incomplete header in pull mode
pulling headers is meant to complete as a whole. If we don't have
enough data, it's an error.

Avoids pipeline hangs on corrupted files

https://bugzilla.gnome.org/show_bug.cgi?id=774846
2016-11-22 18:21:46 +01:00
Edward Hervey
45c7826d76 asfdemux: Handle issues with "empty" files
In some corrupted files, we could end up with no actual streams
being exposed.

In those cases, make sure we properly propagate the failure all
the way to the loop function. This avoids ending up in cases where
we are neither EOS'd nor ERROR'd out from a pipeline point of view.

https://bugzilla.gnome.org/show_bug.cgi?id=774846
2016-11-22 18:21:46 +01:00
Thibault Saunier
3e27368ce3 Use the new API to post flow ERROR messages on the bus
https://bugzilla.gnome.org/show_bug.cgi?id=770158
2016-08-26 19:23:28 -03:00
Thibault Saunier
40dbcd1b3e meson: Link asfdemux against the gstsdp library 2016-08-24 09:45:05 -03:00
Nirbheek Chauhan
c8260a1545 Add support for Meson as alternative/parallel build system
https://github.com/mesonbuild/meson

With contributions from:

Tim-Philipp Müller <tim@centricular.com>
Jussi Pakkanen <jpakkane@gmail.com> (original port)

Highlights of the features provided are:
* Faster builds on Linux (~40-50% faster)
* The ability to build with MSVC on Windows
* Generate Visual Studio project files
* Generate XCode project files
* Much faster builds on Windows (on-par with Linux)
* Seriously fast configure and building on embedded

... and many more. For more details see:

http://blog.nirbheek.in/2016/05/gstreamer-and-meson-new-hope.html
http://blog.nirbheek.in/2016/07/building-and-developing-gstreamer-using.html

Building with Meson should work on both Linux and Windows, but may
need a few more tweaks on other operating systems.
2016-08-20 11:29:37 +01:00
Nirbheek Chauhan
5ae5b2d3c7 plugins: Use <stdint.h> instead of _stdint.h
_stdint.h is generated by Autotools and we don't really need it. All
supported platforms now ship with stdint.h. The only stickler was MSVC,
and since Visual Studio 2015 it also ships stdint.h now.
2016-08-19 13:03:29 +01:00
Tim-Philipp Müller
8149ee2e46 dvdlpcmdec: add libgstbase CFLAGS as well
For consistency.

https://bugzilla.gnome.org/show_bug.cgi?id=769610
2016-08-08 09:58:57 +01:00
xlazom00
eda96ad1c1 dvdlpcmdec: link libgstbase-1.0
https://bugzilla.gnome.org/show_bug.cgi?id=769610
2016-08-08 09:55:31 +01:00
Carlos Rafael Giani
975f147abe asfdemux: Add byte rate based bitrate tag in case there are no audio props
https://bugzilla.gnome.org/show_bug.cgi?id=769418
2016-08-02 14:41:31 +03:00
Michael Olbrich
9f51f72b37 dvdlpcmdec: add support for another format
The format specification can be found here:
http://www.dvdforum.org/images/Guideline1394V10R0_20020911.pdf

Note: This format is also used for Wifi-Display.

https://bugzilla.gnome.org/show_bug.cgi?id=765807
2016-07-25 13:37:32 +03:00
Michael Olbrich
35dc8d0f68 dvdlpcmdec: rewrite to use GstAudioDecoder
https://bugzilla.gnome.org/show_bug.cgi?id=765807
2016-07-25 13:37:24 +03:00
Vootele Vesterblom
6ff1c761e4 asfdemux: fix seeking back after EOS has been reached in push mode
Fix seeking when demuxer is in INDEX state. This happens when we
reached the end of the stream. It should still be possible to do
a flushing seek and seek back to any other position though. Instead
the demuxer would just go straight to EOS again instead of going
back to processing packets again from the new position.

https://bugzilla.gnome.org/show_bug.cgi?id=749066
2016-05-27 17:18:32 +01:00
Guillaume Desmottes
f52343fcf8 rdtmanager: fix clock leak
The clock returned by gst_system_clock_obtain() was never unreffed.

https://bugzilla.gnome.org/show_bug.cgi?id=766565
2016-05-20 12:01:19 +01:00
Vineeth TM
eacdaa56b3 ugly: use new gst_element_class_add_static_pad_template()
https://bugzilla.gnome.org/show_bug.cgi?id=763082
2016-03-24 14:38:13 +02:00
Edward Hervey
dba14178ce asfdemux: Don't modify segment values on accurate seeks
When dealing with accurate seeks, we want to return exactly the segment
that was requested by the seek event, and not adjust the values.

https://bugzilla.gnome.org/show_bug.cgi?id=755469
2016-03-08 12:27:36 +01:00
Tim-Philipp Müller
62f5601723 asfdemux: fix adding bitrate to stream tags
https://bugzilla.gnome.org/show_bug.cgi?id=699586
2016-02-22 22:08:12 +00:00
Tim-Philipp Müller
f1bb44460d win32: remove outdated build cruft
This hasn't been touched for generations, doesn't work,
and is just causing confusion. We also don't want to
maintain these files manually.
2016-02-21 10:11:12 +00:00
Thiago Santos
deb1b257f4 dvdsubdec: use query caps instead of accept-caps
accept-caps does only a shallow check and a full downstream
caps query is needed to check if ARGB is possible
2016-01-08 22:00:12 -03:00
Ravi Kiran K N
9ee60482b2 asfdemux: Support reverse playback
Support reverse playback for ASF format.
Parse packets and queue the payloads, then push
the payload buffer to decoder in reverse order.
Video buffers are pushed from KeyFrame to next
Keyframe.

https://bugzilla.gnome.org/show_bug.cgi?id=757341
2016-01-04 16:13:03 -05:00
Tim-Philipp Müller
d57b097198 dvdlpcmdec: fix invalid read beyond channel position array
We would always copy sizeof(sorted_position) bytes, which is
for 8 channels, but if we have less than 8 channels the
position array we copy from will only have allocated space
for channel channels, so we would read beyond the input
array in some cases.
2015-09-26 09:24:25 +01:00
HoonHee Lee
d7f78391b4 dvdlpcmdec: factor out common code to set output format into new function
When caps event is recieved and header is changed, reordering channel
and setting the default output format for audio are processed. These 2 of
code are same. Thus, It is better to make a new function for these common
code in terms of removing duplicated code, maintenance and expansion.

https://bugzilla.gnome.org/show_bug.cgi?id=753920
2015-09-26 09:18:05 +01:00
Sebastian Dröge
738e48ef49 asfdemux: Don't push any data on pads after the segment.stop
https://bugzilla.gnome.org/show_bug.cgi?id=755469
2015-09-25 23:53:47 +02:00
Sebastian Dröge
155c8fc6a6 asfdemux: Only change segment start to a keyframe for KEYUNIT seeks
Not for accurate seeks, which should accurately start at the requested
position.

https://bugzilla.gnome.org/show_bug.cgi?id=755469
2015-09-25 23:53:47 +02:00
Sebastian Dröge
f501188191 asfdemux: Update segment.position when pushing buffers
Without this, non-flushing seeks are not going to work well.

https://bugzilla.gnome.org/show_bug.cgi?id=755469
2015-09-25 23:53:47 +02:00
Sebastian Dröge
c526153ec9 asfdemux: Only push EOS event if we're not doing a segment seek
https://bugzilla.gnome.org/show_bug.cgi?id=755469
2015-09-25 23:53:47 +02:00
Sebastian Dröge
42d7aec412 asfdemux: If pushing buffers returned EOS, go into the EOS and not failure code path
We don't want to unconditionally send an EOS event downstream, but might want
to send a SEGMENT_DONE message/event instead if a segment seek was done
before.

https://bugzilla.gnome.org/show_bug.cgi?id=755469
2015-09-25 23:53:47 +02:00
Tim-Philipp Müller
b41376aba2 asfdemux: fix file mode of some source files 2015-08-28 19:18:32 +01:00