summaryrefslogtreecommitdiff
path: root/gst/rtp
Commit message (Collapse)AuthorAgeFilesLines
* rtph265depay: make sure we call handle_nal for each NALLuis de Bethencourt2015-08-151-1/+5
| | | | | | | Call handle_nal for each NAL in the STAP-A RTP packet. This makes sure we correctly extract the SPS and PPS. https://bugzilla.gnome.org/show_bug.cgi?id=730999
* rtph265pay: Copy metadata in the payloader, but only the relevant onesLuis de Bethencourt2015-08-151-6/+8
| | | | | | | | The payloader didn't copy anything so far, the depayloader copied every possible meta. Let's make it consistent and just copy all metas without tags or with only the video tag. https://bugzilla.gnome.org/show_bug.cgi?id=751774
* rtph265pay: Use GST_WARNING_OBJECT() instead of GST_WARNING()Luis de Bethencourt2015-08-151-2/+2
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=753228
* rtph265pay: fix potential crash when shutting downLuis de Bethencourt2015-08-151-2/+7
| | | | | | | | | | A race condition in the state change function may cause buffers to be unreffed while they are still used by the streaming thread in gst_rtp_h265_pay_send_vps_sps_pps() resulting in a crash. Chain up to the parent class first in the state change function to make sure streaming has stopped and only then free those buffers. https://bugzilla.gnome.org/show_bug.cgi?id=741381
* rtph265pay: fix buffer leak when using SPS/PPSLuis de Bethencourt2015-08-141-1/+3
| | | | | | | Fixes a buffer leak that would occur if the pipeline was shutdown while a SPS/PPS header was being created. https://bugzilla.gnome.org/show_bug.cgi?id=741271
* rtph265depay: copy metadata in the depayloader, but only the relevant onesLuis de Bethencourt2015-08-142-0/+89
| | | | | | | | The payloader didn't copy anything so far, the depayloader copied every possible meta. Let's make it consistent and just copy all metas without tags or with only the video tag. https://bugzilla.gnome.org/show_bug.cgi?id=751774
* rtph265depay: checking if depay has sps/pps nals before insertionLuis de Bethencourt2015-08-121-1/+1
| | | | | | Related to: https://bugzilla.gnome.org/show_bug.cgi?id=753430 https://bugzilla.gnome.org/show_bug.cgi?id=753228
* rtph265depay: only update the srcpad caps if something else than the ↵Luis de Bethencourt2015-08-121-2/+41
| | | | | | | | | | codec_data changed h264parse and gstrtph264depay do the same, let's keep the behaviour consistent. As we now include the codec_data inside the stream, this causes less caps renegotiation. https://bugzilla.gnome.org/show_bug.cgi?id=753228
* rtph265depay: PPS replaces old PPS if it has the same idLuis de Bethencourt2015-08-121-1/+1
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=753228
* rtph265depay: Insert SPS/PPS NALs into the streamLuis de Bethencourt2015-08-121-0/+50
| | | | | | | | | rtph264depay does the same and this fixes decoding of some streams with 32 SPS (or 256 PPS). It is allowed to have SPS ID 0 to 31 (or PPS ID 0 to 255), but the field in the codec_data for the number of SPS or PPS is only 5 (or 8) bit. As such, 32 SPS (or 256 PPS) are interpreted as 0 everywhere. This looks like a mistake in the part of the spect about the codec_data.
* rtph265depay: implement process_rtp_packet() vfuncLuis de Bethencourt2015-08-121-16/+8
| | | | | | | | | | For more optimised RTP packet handling: means we don't need to map the input buffer again but can just re-use the mapping the base class has already done. Based on: https://bugzilla.gnome.org/show_bug.cgi?id=750235 https://bugzilla.gnome.org/show_bug.cgi?id=753228
* rtph265depay: Use GST_BUFFER_PTS() instead of GST_BUFFER_TIMESTAMP()Luis de Bethencourt2015-08-121-2/+2
| | | | Switching to GST_BUFFER_TIMESTAMP() to be consistent with other rtp code.
* rtph265depay: prevent trying to get 0 bytes from adapterLuis de Bethencourt2015-08-121-4/+6
| | | | | | | | This causes an assertion and would lead to getting a NULL instead of a buffer. Without proper checking this would easily lead to a segfault. Related to rpth264depay: https://bugzilla.gnome.org/show_bug.cgi?id=737199
* rtp: remove dead assignmentLuis de Bethencourt2015-07-291-2/+0
| | | | | Value set to ret will be overwritten at least once at the end of the while loop, removing assignment.
* Remove obsolete Android build cruftTim-Philipp Müller2015-04-261-14/+0
| | | | This is not needed any longer.
* remove unused enum items PROP_LASTLuis de Bethencourt2015-04-241-2/+1
| | | | | This were probably added to the enums due to cargo cult programming and are unused.
* rtp: donl_present variable unusedLuis de Bethencourt2015-03-061-1/+11
| | | | | | | donl_present is not implemented, yet the value is set and checked a few times. Cleaning this. CID #1249687
* rtp: value truncated too short creates dead codeLuis de Bethencourt2015-01-081-1/+1
| | | | | | | | type is truncated to 0-31 with "& 0x1f", but right after that it is checks if the value is equivalent to GST_H265_NAL_VPS, GST_H265_NAL_SPS, and GST_H265_NAL_PPS (which are 32, 33, and 34 respectively). Obviously, this will never be True if the value is maximum 31 after the truncation. The intention of the code was to truncate to 0-63.
* rtp: fix nal unit type checkLuis de Bethencourt2015-01-081-1/+2
| | | | | | After further investigation the previous commit is wrong. The code intended to check if the type is 39 or the ranges 41-44 and 48-55. Just like gsth265parse.c does. Type 40 would not be complete.
* rtp: fix dead code and check for impossible valuesLuis de Bethencourt2015-01-081-2/+1
| | | | | | | | | | | | | | | nal_type is the index for a GstH265NalUnitType enum. There are two types of dead code here: First, after checking if nal_type is >= 39 there are two OR conditionals that check if the value is in ranges higher than that number, so if nal_type >= 39 falls in the True branch those other conditions aren't checked and if it falls in the False branch and they are checked, they will always also be False. They are redundant. Second, the enum has a range of 0 to 40. So the checks for ranges higher than 41 should never be True. Removing this redundant checks. CID 1249684
* rtpbad: Fix make dist by removing non-existing file from EXTRA_DISTSebastian Dröge2014-10-191-2/+0
| | | | | It's used by the rtp plugin in gst-plugins-good for VP8 but not needed or used here.
* rtpbad: include plugins base cflags (fix build on jenkins)Thijs Vermeir2014-10-171-1/+1
|
* rtp: add h265 RTP payloader + depayloaderThijs Vermeir2014-10-176-0/+3168
View Lorry Controller Status