Sebastian Dröge
fd261e1099
aacparse: Reject raw AAC if no codec_data is found in the caps
...
If necessary, a demuxer will have to invent something here but this is only a
problem with non-conformant files anyway.
2016-07-04 16:58:38 +02:00
Sebastian Dröge
df454fa28f
qtdemux: Invent AAC codec_data if none is present
...
Without, raw AAC can't be handled and we have some information available in
the decoder that most likely allows us to decode the stream in one way or
another. This is the same code already used by matroskademux for the same
reasons, and ffmpeg/vlc play such files just fine too by guesswork.
2016-07-04 16:55:32 +02:00
Sebastian Dröge
5b24841f66
qtmux: Reject raw AAC caps without codec_data
...
The resulting file is not going to be playable without guesswork and raw caps
should always have codec_data.
2016-07-04 14:54:13 +02:00
Edward Hervey
e3923df800
qtdemux: Handle upstream GAP in push-mode/time segment
...
This is to handle cases where upstream handles the fragmented streaming in TIME
segments and sends us data with gaps within fragments. This would happen when dealing
with trick-modes.
When upstream (push-based, TIME SEGMENT) wishes to send discontinuous samples,
it must obey the following rules:
* The buffer containing the [moof] must have a valid GST_BUFFER_OFFSET
* The buffers containing the first sample after a gap:
* MUST start at the beginning of a sample,
* MUST have the DISCONT flag set,
* MUST have a valid GST_BUFFER_OFFSET relative to the beginning of the fragment.
https://bugzilla.gnome.org/show_bug.cgi?id=767354
2016-07-01 14:21:04 +02:00
Brad Lackey
6d3071f200
rtspsrc: Don't disable UDP protocols on redirecting
...
https://bugzilla.gnome.org/show_bug.cgi?id=768232
2016-07-01 12:21:43 +02:00
Seungha Yang
231018bcfe
qtdemux: Push caps only when it was updated
...
Commit 7873bede31
caused new caps
event per moof without consideration of duplication.
https://bugzilla.gnome.org/show_bug.cgi?id=768268
2016-07-01 11:37:20 +02:00
Jonas Holmberg
850a8bc077
rtph265depay: fix invalid memory access
...
10 bytes was allocated for stream_format but size of "byte-stream" is
more. Use g_strdup() instead.
https://bugzilla.gnome.org/show_bug.cgi?id=753760
2016-06-30 16:56:24 +01:00
Sebastian Dröge
75963b47f4
udpsrc: Windows has no ipi_spec_dst in struct in_pktinfo
2016-06-28 16:44:50 +03:00
Sebastian Dröge
cdd5fa4d96
udpsrc: #define __APPLE_USE_RFC_3542 to be able to use IPV6_PKTINFO on OSX/iOS
2016-06-28 15:15:14 +03:00
Sebastian Dröge
36a154fa96
udpsrc: Move #includes around to a) work around broken glibc header and b) Windows
2016-06-28 15:08:04 +03:00
Sebastian Dröge
7e47579f17
udpsrc: Fix compilation on Windows and *BSD/OSX
2016-06-28 14:25:03 +03:00
Sebastian Dröge
123d62712c
udpsrc: Filter out multicast packets that are not for our multicast address
...
https://bugzilla.gnome.org/show_bug.cgi?id=767980
2016-06-28 13:40:06 +03:00
Sebastian Dröge
c18b609c06
rtspsrc: When seeking, consider the current element state or pending state instead of the RTSP state
...
If we consider the RTSP state, what can happen is that it is PLAYING but the
element already asynchronously tried to PAUSE and it just did not happen yet.
We would then override this setting to PAUSED (while the element actually is
in PAUSED) and set the RTSP state to PLAYING again. This would then cause us
to produce packets while the sinks are all PAUSED, piling up thousands of
packets in the rtpjitterbuffer and other elements and finally failing.
2016-06-28 11:01:24 +03:00
Sebastian Dröge
d6f597db20
flvdemux: Add comment about H263/MPEG4P2 being non-standard for FLV
...
They are however supported by ffmpeg and apparently used out there.
https://bugzilla.gnome.org/show_bug.cgi?id=768006
2016-06-27 09:20:35 +03:00
Vivia Nikolaidou
6ac02f8595
flvdemux: Add support for H263 and MPEG4 part2
...
https://bugzilla.gnome.org/show_bug.cgi?id=768006
2016-06-24 15:30:03 +03:00
Aaron Boxer
f07c704b49
gstrtpj2kpay: use tile bit and tile number to determine if there are multiple tiles in packet
...
Now we don't have to rely on a special value for the tile number.
https://bugzilla.gnome.org/show_bug.cgi?id=767817
2016-06-21 13:03:09 +01:00
Tim-Philipp Müller
323244bc04
rtpj2kpay: fix compiler warning on OS/X
...
gstrtpj2kpay.c:364:21: error: implicit truncation from 'int' to bitfield changes value from -1 to 65535
https://bugzilla.gnome.org/show_bug.cgi?id=767817
2016-06-21 09:34:56 +01:00
Sebastian Dröge
5f2b32e642
rtph264pay: Deprecated sprop-parameter-set property
...
This is supposed to be either in the codec_data (avc stream format) or inside
the stream, and we extract it from there. It should not be set from a
property as it's stream specific.
https://bugzilla.gnome.org/show_bug.cgi?id=767789
2016-06-21 10:03:04 +03:00
Aleix Conchillo Flaqué
12eb5d6912
rtspsrc: make all srtp encoder properties explicit
...
The Session Data Protocol doesn't allow specifying a cipher for the
SRTCP, so it will use the SRTP one. In the "srtpenc" element the cipher
"aes-128-icm" is the default for SRTP and SRTCP, but if we want to have
an SRTCP with the "aes-256-icm" cipher then we also need to set the SRTP
cipher to "aes-256-icm", otherwise "aes-128-icm" will be used instead.
https://bugzilla.gnome.org/show_bug.cgi?id=767799
2016-06-20 09:53:24 +02:00
Sebastian Dröge
5a7217a147
qtmux: The prores variant is stored in the variant field, not format
...
And the caps in the sink pad template already used variant (only).
2016-06-17 16:08:08 +03:00
Jonas Holmberg
83ec89abdd
rtph265pay: Remove sprop-parameter-sets property
...
There is no valid use case when this property is needed since the values
must be in either codec_data or buffer data.
https://bugzilla.gnome.org/show_bug.cgi?id=753760
2016-06-17 15:25:57 +03:00
Jonas Holmberg
2039e0d881
rtph265pay: Read NALU type the same way everywhere
...
Cosmetic change to read NALU type in gst_rtp_h265_pay_decode_nal() the
same way as in other places.
https://bugzilla.gnome.org/show_bug.cgi?id=753760
2016-06-17 15:25:57 +03:00
Aurélien Zanelli
f8f8935c77
rtpjitterbuffer: fix RTPJitterBufferMode documentation
...
Documentation lacks '@' before each enum values and there was an extra
line after symbol section which confuses GTK-Doc parser.
https://bugzilla.gnome.org/show_bug.cgi?id=767788
2016-06-17 15:16:45 +03:00
Miguel París Díaz
83f4c08747
rtpsession: take the lock when changing stats
...
https://bugzilla.gnome.org/show_bug.cgi?id=766025
2016-06-17 12:52:29 +03:00
Jürgen Slowack
98b62e397b
rtph265: fix NAL unit type parsing and SPS/PPS/VPS detection
...
Fixes sps/pps/vps insertion via the config-interval property.
https://bugzilla.gnome.org//show_bug.cgi?id=767680
2016-06-15 13:10:50 +01:00
Tim-Philipp Müller
51a0dc2df2
flvdemux: fix indentation
2016-06-10 13:51:39 +01:00
Tim-Philipp Müller
c51831a245
flvdemux: fix date parsing when there are trailing spaces
...
Fixes parsing of "Thu May 11 15:57:46 2006 ".
https://bugzilla.gnome.org/show_bug.cgi?id=767496
2016-06-10 13:51:39 +01:00
Aaron Boxer
b4a4fa19a1
gstrtpj2k: set sampling field required by RFC
...
This field is now required in the sink caps.
https://bugzilla.gnome.org/show_bug.cgi?id=766236
2016-06-10 13:14:44 +03:00
Seungha Yang
4e23d206b9
flvdemux: Fix unref assertion failure
...
Fix unref assertion failure
https://bugzilla.gnome.org/show_bug.cgi?id=767424
2016-06-08 22:01:11 -04:00
Olivier Crête
5328378132
rtpjitterbuffer: Work with non-TIME segments
...
With non-time segments, it now assumes that the arrival time of packets
is not relevant and that only the RTP timestamp matter and it produces
an output segment start at running time 0.
https://bugzilla.gnome.org/show_bug.cgi?id=766438
2016-06-08 14:49:49 -04:00
Edward Hervey
30d2918ab0
qtdemux: Show state name in debugging
...
Makes it easier to trace what's going on
2016-06-07 18:40:14 +03:00
Edward Hervey
7d309d3f4b
qtdemux: Remove useless variable
...
That variable is only needed for a debug statement, move it there
2016-06-07 18:40:14 +03:00
Edward Hervey
d8f1a6c58e
qtdemux: Add/Fix comments on the various structure variables
...
No variables were added/removed. This was just a good excuse to:
* Comment what most variables are used for (and when)
* Order them in such a way as to show first the common variables used
in all cases, followed by those only used in push-mode
2016-06-07 18:40:14 +03:00
Edward Hervey
6f1eed7f02
qtdemux: Remove unused structure
...
Let's just remove it, been commented for 7+ years :)
2016-06-07 18:40:14 +03:00
Sebastian Dröge
24862c2f74
qtdemux: Forward segments directly if we are operating in PUSH mode on fragmented streams
...
We shouldn't go through segment activation as we will only have a limited
understanding of how the whole stream timeline looks like from the moof. We
only know about the current fragment, while upstream knows about the whole
stream.
This fixes seeking in DASH streams, both for seeks after the current moof and
for seeks into the current moof. The former would fail because the moof ends
and we can't activate any segment, the latter would cause a segment that stops
at the moof end, and no further fragments would be played because we end up
being EOS.
https://bugzilla.gnome.org/show_bug.cgi?id=767071
2016-06-07 16:19:39 +03:00
Michael Olbrich
c5da4dc66a
matroskademux: preserve seek flags
...
Without this some flags get lost in streaming mode.
https://bugzilla.gnome.org/show_bug.cgi?id=767194
2016-06-06 10:50:02 +03:00
Miguel París Díaz
389e0abeb0
rtpsource: complete warn log with SSRC
...
https://bugzilla.gnome.org/show_bug.cgi?id=767195
2016-06-06 10:47:17 +03:00
Olivier Crête
91a2a790e9
rtpvp9depay: Don't assert on flexible mode packets
...
Instead just post a warning on the bus for now.
2016-06-02 16:17:19 -04:00
Edward Hervey
1d2db2ba4f
deinterlace: Ensure DISCONT flag is properly propagated
...
The output of deinterlace at startup, or when receiving a new DISCONT
buffer, should have the DISCONT flag set on the first buffer.
2016-06-02 11:35:27 +03:00
Sebastian Dröge
4498e57c10
qtdemux: Use the demuxer segment instead of a new one for MSS streams
...
Upstream might have told us something about the to be expected segment, so
let's use that information instead of coming up with a [0,-1] segment.
https://bugzilla.gnome.org/show_bug.cgi?id=767071
2016-06-01 09:32:03 +03:00
Sebastian Dröge
84e698c531
qtdemux: Only activate segments and send SEGMENT events if we have streams
...
But in that case also remove the pending newsegment event, otherwise we would
later send a possibly outdated event.
https://bugzilla.gnome.org/show_bug.cgi?id=767071
2016-06-01 09:32:03 +03:00
Sebastian Dröge
f8eb909d90
qtdemux: In PULL mode, nothing is ever going to send us a SEGMENT event
...
https://bugzilla.gnome.org/show_bug.cgi?id=767071
2016-06-01 09:32:03 +03:00
Sebastian Dröge
f3e68164e4
qtdemux: Don't override TIME segments from upstream that we just saw
...
The point of d8fb7a9c96
was to not have any
spurious segments stored for later if we do BYTES->TIME conversion, but
overriding any TIME segments from upstream does not make any sense.
See https://bugzilla.gnome.org/show_bug.cgi?id=763165
https://bugzilla.gnome.org/show_bug.cgi?id=767071
2016-06-01 09:32:03 +03:00
Prashant Gotarne
4bdd192fb3
multifilesrc: set position as offset from start-index
...
query position in GST_FORMAT_BUFFER returns
offset from start-index rather than index.
https://bugzilla.gnome.org/show_bug.cgi?id=752462
2016-05-27 20:32:08 +01:00
Pierre Lamot
3c50fd7669
rtpj2kpay: Fix buffer memory leak
...
Input buffer memory was not unmapped
https://bugzilla.gnome.org/show_bug.cgi?id=766870
2016-05-27 12:46:23 +01:00
Tim-Philipp Müller
3d979d4e87
videocrop mark crop properties as mutable in playing state
2016-05-23 19:17:08 +01:00
Sebastian Dröge
7cd9d34c80
qtdemux: Set seek event seqnum on all SEGMENT events
...
Some were forgotten.
See https://bugzilla.gnome.org/show_bug.cgi?id=765935
2016-05-20 11:15:44 +03:00
Sebastian Dröge
9e5cda59f8
avidemux: Pass through seek event seqnums in all SEGMENT/EOS events and SEGMENT_DONE messages/events
...
See https://bugzilla.gnome.org/show_bug.cgi?id=765935
2016-05-20 11:12:44 +03:00
Sebastian Dröge
0345ba78f5
matroskademux: Set seek event seqnum in EOS and SEGMENT_DONE messages/events
...
Also actually store the seqnum in pull mode seeks.
See https://bugzilla.gnome.org/show_bug.cgi?id=765935
2016-05-20 10:57:30 +03:00
Guillaume Desmottes
47a358783e
deinterlace: fix caps leak
...
The caps returned by gst_pad_get_current_caps() was never unreffed when
not early returning.
Fix a leak with the elements/deinterlace test.
https://bugzilla.gnome.org/show_bug.cgi?id=766558
2016-05-20 09:36:09 +03:00