libav.git
2 years agoUpdate changelog for v12.2 v12.2
Sean McGovern [Sat, 30 Sep 2017 17:32:20 +0000 (13:32 -0400)]
Update changelog for v12.2

2 years agosmacker: fix integer overflow with pts_inc
Michael Niedermayer [Sun, 24 Sep 2017 10:58:42 +0000 (12:58 +0200)]
smacker: fix integer overflow with pts_inc

Bug-Id: 1073

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
(cherry picked from commit ec683ed527cef9aad208d1daeb10d0e7fb63e75e)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agosmacker: return meaningful error codes on failure
Sean McGovern [Sun, 24 Sep 2017 10:57:54 +0000 (12:57 +0200)]
smacker: return meaningful error codes on failure

(cherry picked from commit 9e361022782514d4d4ce97b9c9d5a7863a2de519)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoarm: Fix SIGBUS on ARM when compiled with binutils 2.29
Martin Storsjö [Thu, 31 Aug 2017 09:06:44 +0000 (12:06 +0300)]
arm: Fix SIGBUS on ARM when compiled with binutils 2.29

In binutils 2.29, the behavior of the ADR instruction changed so that 1 is
added to the address of a Thumb function (previously nothing was added). This
allows the loaded address to be passed to a BLX instruction and the correct
mode change will occur.

See: https://sourceware.org/bugzilla/show_bug.cgi?id=21458

By using adr with a label that isn't annotated as a thumb function,
we avoid the new behaviour in binutils 2.29 and get the same behaviour
as in prior releases, and as in other assemblers (ms armasm.exe,
clang's built in assembler) - an idea that Janne Grunau came up with.

Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 9dde6ab06c48f9447cd16f39bee33569cddb7be4)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agomov: move stsd finalization to an appropriate place
Sean McGovern [Sat, 29 Jul 2017 23:17:16 +0000 (19:17 -0400)]
mov: move stsd finalization to an appropriate place

mov_finalize_stsd_codec() parses stream information from the ALAC extradata,
so run it after the extradata processing is completed in mov_read_stsd().

Fixes playback of 96kHz ALAC streams muxed by qaac or the reference alac encoder.

Adapted from an FFmpeg patch by Hendrik Leppkes <h.leppkes@gmail.com>

Bug-Id: 1072
(cherry picked from commit defe307fb22beca60a632e976ab97e5edd4aee25)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agomov: Do not set stsd_count if mov_read_stsd() fails
Sean McGovern [Fri, 28 Jul 2017 20:29:35 +0000 (16:29 -0400)]
mov: Do not set stsd_count if mov_read_stsd() fails

Based on an FFmpeg patch by Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 3050dabaa9a337ad077ec60bba664ad9861e1aa6)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agomov: log and return early on non-positive stsd entry counts
Sean McGovern [Fri, 28 Jul 2017 20:17:33 +0000 (16:17 -0400)]
mov: log and return early on non-positive stsd entry counts

Based on an FFmpeg patch by Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit d7bdab1ad78ef582ba8c96dc7b79ec9fdbeeb94f)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoyadif: Account for the buffer alignment while processing the frame edges
Michael Niedermayer [Mon, 21 Aug 2017 07:55:48 +0000 (09:55 +0200)]
yadif: Account for the buffer alignment while processing the frame edges

Avoid out of bound reads.

Bug-Id: 1031
CC: libav-stable@libav.org
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit feed239021bad89743d5e7989b426ae594322eb7)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agoh264dec: use a large enough field for reference list modification values
Anton Khirnov [Thu, 17 Aug 2017 10:15:58 +0000 (12:15 +0200)]
h264dec: use a large enough field for reference list modification values

pic_num can be at most 17-bit, so uint8_t is not sufficient.

Found-By: Bradley Sepos <bradley@bradleysepos.com>
CC: libav-stable@libav.org
(cherry picked from commit f70f71d60c7ae88c19078a48dc6e0789b78c7300)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoh264dec: track the last seen value of x264_build
Anton Khirnov [Sat, 22 Jul 2017 12:52:27 +0000 (14:52 +0200)]
h264dec: track the last seen value of x264_build

Do not use the one in the SEI directly as that is reset at certain
points.

Inspired by patches from Michael Niedermayer <michaelni@gmx.at> and
Anton Mitrofanov <BugMaster@narod.ru>.

CC: libav-stable@libav.org
(cherry picked from commit 45c4bf3df03ef53ae61fa1473424d4ae024f22e4)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh264dec: fix Lossless Decoding (Profile 244) for 8x8 Intra Prediction
Yogender Kumar Gupta [Thu, 15 Jun 2017 22:56:13 +0000 (18:56 -0400)]
h264dec: fix Lossless Decoding (Profile 244) for 8x8 Intra Prediction

CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 79c6477c2abd8cfa41eef0c4ac39779dd8a9ec8e)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh264_cabac: Fix CABAC+8x8dct in 4:4:4
Anton Mitrofanov [Thu, 15 Jun 2017 22:56:14 +0000 (18:56 -0400)]
h264_cabac: Fix CABAC+8x8dct in 4:4:4

Use the correct ctxIdxInc calculation for coded_block_flag.
Keep old behavior for old versions of x264 for backward compatibility.

CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 18d3f36d3c4d0f2c3e702f970ff8b457d7d5e39c)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh264dec: Fix mix of lossless and lossy MBs decoding
Anton Mitrofanov [Thu, 15 Jun 2017 22:56:16 +0000 (18:56 -0400)]
h264dec: Fix mix of lossless and lossy MBs decoding

CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 70946e605924e2108c39f96faa369c220177f301)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agocaf: add an Opus tag
Anton Khirnov [Sat, 1 Jul 2017 08:49:58 +0000 (10:49 +0200)]
caf: add an Opus tag

CC: libav-stable@libav.org
(cherry picked from commit 48a5c35346aeded1c65b8709bbb085fa4e705c91)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agormdec: don't ignore the return value of av_get_packet()
Sean McGovern [Wed, 12 Jul 2017 23:45:46 +0000 (19:45 -0400)]
rmdec: don't ignore the return value of av_get_packet()

(cherry picked from commits 95ce02b35d3d1bb16111031df1d82a6e4d894d36i
 and 80e919b17435da18c0f2be6403a0315cf4b40a1a)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agodfa: Disallow odd width/height and add proper bounds check for DDS1 chunks
Diego Biurrun [Fri, 11 Aug 2017 17:15:20 +0000 (19:15 +0200)]
dfa: Disallow odd width/height and add proper bounds check for DDS1 chunks

DDS1 chunks are decoded in 2x2 blocks, odd chunk width or height is not
allowed in that case. Also ensure that the decode buffer is big enough
for all blocks being processed.

Bug-Id: CVE-2017-9992
CC: libav-stable@libav.org
(cherry picked from commit d34a133b78afe2793cd8537f3c7f42437f441e94)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agodoc: Drop the legacy symlink to README
Luca Barbato [Tue, 13 Jun 2017 23:40:40 +0000 (01:40 +0200)]
doc: Drop the legacy symlink to README

Windows does not like symlinks and README.md is pretty common nowadays.

(cherry picked from commit c5c76370ae240b5e451c4a00a7ab485c0c0e0e98)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agodvbsubdec: Fixed segfault when decoding subtitles
Lorenz Brun [Fri, 21 Oct 2016 20:51:37 +0000 (22:51 +0200)]
dvbsubdec: Fixed segfault when decoding subtitles

This fixes a segfault (originally found in Movian, but traced to libav)
when decoding subtitles because only an array of rects is allocated,
but not the actual structs it contains. The issue was probably
introduced in commit 2383323 where the loop to allocate the rects in
the array was thrown away.

Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
(cherry picked from commit 1cfd566324f4a9be066ea400685b81c0695e64d9)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agortmpdh: Don't use the OpenSSL DH struct
Martin Storsjö [Fri, 21 Oct 2016 09:40:05 +0000 (12:40 +0300)]
rtmpdh: Don't use the OpenSSL DH struct

Instead use our own struct, which we already use when using
gcrypt and gnutls.

In OpenSSL 1.1, the DH struct has been made opaque.

Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 016387fe0fe3eff1a03ec0673bf4d2967f6cad94)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agoopenssl: Support version 1.1.0.
Matt Oliver [Sun, 9 Oct 2016 19:49:54 +0000 (06:49 +1100)]
openssl: Support version 1.1.0.

Further simplifications by Martin Storsjö, to minimize the
diff.

Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit ee050797664c7c74cae262ffab05006b55d47a11)
(cherry picked from commit f22363c72968f1a1fc4881d8695ec7068b0aa03c)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agoconfigure: Simplify and fix libxcb check
Diego Biurrun [Fri, 23 Jun 2017 10:47:10 +0000 (10:47 +0000)]
configure: Simplify and fix libxcb check

Check for xcb as well as xcb-shape before enabling libxcb since newer
versions of libxcb have xcb-foo pkg-config files that do not declare
their xcb dependency so that required linker flags will not be generated.

Use helper functions to simplify libxcb check, drop unused variables.

(cherry picked from commit 1ea77aae927c7310034b1f75d4f1c2676fe641f2)
(cherry picked from commit a97563c889fefd81ad6b3758471434d8c2e2e550)
(cherry picked from commit 871b4f3654636ed64560e86b9faa33828d195ceb)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agobuild: Add an option for passing linker flags to the shared library build
Janne Grunau [Tue, 20 Jun 2017 17:25:43 +0000 (19:25 +0200)]
build: Add an option for passing linker flags to the shared library build

Also employ this mechanism to pass $libdir to the runtime library search
path if rpath is enabled. This fixes underlinking of some test binaries
on some systems.

(cherry picked from commit 857e26b655a769e5a56bada1a0d9adb44cc176b7)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agosmacker: Check that the data size is a multiple of a sample vector
Michael Niedermayer [Sun, 15 Nov 2015 13:52:08 +0000 (14:52 +0100)]
smacker: Check that the data size is a multiple of a sample vector

Fixes out of array access
Fixes: ce19e41f0ef1e52a23edc488faecdb58/asan_heap-oob_2504e97_4202_ffa0df1baed14022b9bfd4f8ac23d0cb.smk

Bug-Id: CVE-2015-8365
CC: libav-stable@libav.org
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 4a9af07a49295e014b059c1ab624c40345af5892)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
(cherry picked from commit b98f082d8ddc0a0d8317114d8414ab51de60ef02)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
2 years agompegvideo_motion: Handle edge emulation even without unrestricted_mv
Michael Niedermayer [Tue, 12 Nov 2013 15:11:42 +0000 (16:11 +0100)]
mpegvideo_motion: Handle edge emulation even without unrestricted_mv

Fix out of bounds read.

Bug-Id: 962, 1060
CC: libav-stable@libav.org
Found by: F4B3CD@STARLAB and Agostino Sarubbo
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
(cherry picked from commit 136f55207521f0b03194ef5b55ba70f1635d6aee)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoUpdate changelog for v12.1 v12.1
Sean McGovern [Sun, 14 May 2017 18:14:56 +0000 (14:14 -0400)]
Update changelog for v12.1

2 years agoh264_refs: validate the SPS pointer in ff_h264_execute_ref_pic_marking()
Sean McGovern [Sun, 7 May 2017 21:11:59 +0000 (17:11 -0400)]
h264_refs: validate the SPS pointer in ff_h264_execute_ref_pic_marking()

Bug-Id: 1036
CC: libav-stable@libav.org
(cherry picked from commit cb167f2947f1a2c446bd8db196d0e64ef4a6d06b)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoaacsbr: Turnoff in the event of over read.
Alex Converse [Mon, 20 Mar 2017 19:44:42 +0000 (12:44 -0700)]
aacsbr: Turnoff in the event of over read.

Aliased compressed AAC bytes are almost certainly not meaningful SBR
data. In the wild this causes harsh artifacts switching HE-AAC streams
that don't have SBR headers aligned with segment boundaries.

Turning off SBR falls back to a default set of upsampling parameters
that can function as a sort of error concealment. This is consistent
with how the decoder handles other sorts of errors.

Bug-Id: 1047
CC: libav-stable@libav.org
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoavprobe: Handle unknown values for the color description
Luca Barbato [Mon, 17 Apr 2017 13:48:54 +0000 (13:48 +0000)]
avprobe: Handle unknown values for the color description

print_str() cannot print NULL.

Bug-Id: 1040
CC: libav-stable@libav.org
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit c55f2ae2b8d4de473f945aebd6a7a89b6b83e87a)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoavplay: Do not try to allocate new frames when the player is closing
Luca Barbato [Mon, 17 Apr 2017 22:22:17 +0000 (22:22 +0000)]
avplay: Do not try to allocate new frames when the player is closing

The allocation event can trigger while the decoding thread is already
closing.

Bug-Id: 1052
CC: libav-stable@libav.org
(cherry picked from commit 8c0cadd17e98547d84e82111550caca4fb40ff8d)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agovc1dec: raise an error if sprite picture data is missing
Sean McGovern [Sat, 22 Apr 2017 23:16:34 +0000 (19:16 -0400)]
vc1dec: raise an error if sprite picture data is missing

Bug-Id: 1013
CC: libav-stable@libav.org
(cherry picked from commit 2e1ab6a67dd5d9a30558d008aa6e541c68d76221)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agomm: Skip unexpected audio packets
Luca Barbato [Mon, 17 Apr 2017 19:25:35 +0000 (19:25 +0000)]
mm: Skip unexpected audio packets

Bug-Id: 1046
CC: libav-stable@libav.org
(cherry picked from commit 1731c3530bffb876deb9e00dfffdf9841a8412cd)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agomjpeg: Report non-3 component rgb lossless as not supported
Luca Barbato [Mon, 17 Apr 2017 19:11:57 +0000 (19:11 +0000)]
mjpeg: Report non-3 component rgb lossless as not supported

Bug-Id: 1043
CC: libav-stable@libav.org
(cherry picked from commit f2c469b73f8ac3dd6a98d38281f61b68ea6dd336)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoflv: Validate the packet size
Luca Barbato [Mon, 17 Apr 2017 14:10:52 +0000 (14:10 +0000)]
flv: Validate the packet size

Size can be negative at that point.

Bug-Id: 1041
CC: libav-stable@libav.org
(cherry picked from commit 279e3aaa14daba6b7a37d75f3fb6e29c732d123f)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agodca: Account for lfe when checking for the channel count
Luca Barbato [Mon, 17 Apr 2017 12:53:14 +0000 (12:53 +0000)]
dca: Account for lfe when checking for the channel count

Bug-Id: 1037
CC: libav-stable@libav.org
(cherry picked from commit 5352802da81f2083e65d466612e639a4e6e5530e)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoavformat: Free the internal codec context at the end
Luca Barbato [Tue, 11 Apr 2017 23:46:30 +0000 (01:46 +0200)]
avformat: Free the internal codec context at the end

Avoid a use after free in avformat_find_stream_info.

CC: libav-stable@libav.org
(cherry picked from commit 9e4a5eb51b9f3b2bff0ef08e0074b7fe4893075d)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agodca: Validate the channel map
Luca Barbato [Tue, 14 Mar 2017 09:15:29 +0000 (09:15 +0000)]
dca: Validate the channel map

Having a mismatch between the number of channels in the stream and those
in the channel map will lead to a segfault or worse.

Bug-Id: 1016

CC: libav-stable@libav.org
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit e245d4f45ca5e2b97daef5944d63323c07d545bc)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agompeg4videodec: raise an error if sprite_trajectory.table is NULL
Sean McGovern [Thu, 30 Mar 2017 20:21:38 +0000 (16:21 -0400)]
mpeg4videodec: raise an error if sprite_trajectory.table is NULL

CC: libav-stable@libav.org
Bug-Id: 1012
(cherry picked from commit 6ac0e7818399a57e4684202bac79f35b3561ad1e)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agonsvdec: don't ignore the return value of av_get_packet()
Sean McGovern [Fri, 24 Mar 2017 17:36:51 +0000 (13:36 -0400)]
nsvdec: don't ignore the return value of av_get_packet()

Fixes invalid reads with corrupted files.

CC: libav-stable@libav.org
Bug-Id: 1039
(cherry picked from commit fe6eea99efac66839052af547426518efd970b24)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh264_cavlc: check the value of run_before
Anton Khirnov [Wed, 28 Dec 2016 12:02:02 +0000 (13:02 +0100)]
h264_cavlc: check the value of run_before

Section 9.2.3.2 of the spec implies that run_before must not be larger
than zeros_left.

Fixes invalid reads with corrupted files.

CC: libav-stable@libav.org
Bug-Id: 1000
Found-By: Kamil Frankowicz
(cherry picked from commit 522d850e68ec4b77d3477b3c8f55b1ba00a9d69a)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh2645_parse: use the bytestream2 API for packet splitting
Anton Khirnov [Wed, 28 Dec 2016 10:27:56 +0000 (11:27 +0100)]
h2645_parse: use the bytestream2 API for packet splitting

The code does some nontrivial jumping around in the buffer, so it is
safer to use a checked API rather than do everything manually.

Fixes a bug in nalff parsing, where the length field is currently not
counted in the buffer size check, resulting in possible overreads with
invalid files.

CC: libav-stable@libav.org
Bug-Id: 1002
Found-By: Kamil Frankowicz
(cherry picked from commit 83b2b34d06e74cc8775ba3d833f9782505e17539)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoh264dec: initialize field_started to 0 on each decode call
Anton Khirnov [Wed, 28 Dec 2016 10:05:25 +0000 (11:05 +0100)]
h264dec: initialize field_started to 0 on each decode call

It might be incorrectly set to 1 if the previous call exited with an
error.

Bug-Id: 1019
CC: libav-stable@libav.org
(cherry picked from commit b76f6a76c6312dc551d7c37c6ded36bea7973c74)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agolavc: make sure not to return EAGAIN from codecs
Anton Khirnov [Tue, 14 Feb 2017 19:51:06 +0000 (20:51 +0100)]
lavc: make sure not to return EAGAIN from codecs

This error is treated specially by the API.

CC: libav-stable@libav.org
(cherry picked from commit 984736dd9e5b50987a5910e22495304e4a6d975c)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoapetag: account for header size if present when returning the start position
James Almer [Fri, 10 Feb 2017 23:24:27 +0000 (20:24 -0300)]
apetag: account for header size if present when returning the start position

The size field in the header/footer accounts for the entire APE tag
structure except the 32 bytes from header, for compatibility with
APEv1.

Signed-off-by: James Almer <jamrial@gmail.com>
CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 4cc0227040adb9efc63be6a5765e3214f5c6f662)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoapetag: fix flag value to signal footer presence
James Almer [Fri, 10 Feb 2017 23:24:26 +0000 (20:24 -0300)]
apetag: fix flag value to signal footer presence

According to the spec[1], a value of 0 means the footer is present and a value
of 1 means it's absent, the exact opposite of header presence flag where 1
means present and 0 absent.
The reason for this is compatibility with APEv1 tags, where there's no header,
footer presence was mandatory for all files, and the flags field was a zeroed
reserved field.

[1] http://wiki.hydrogenaud.io/index.php?title=Ape_Tags_Flags

Signed-off-by: James Almer <jamrial@gmail.com>
CC: libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 3f258f5ee05c9da05f61447b802ae3e39629f44b)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agortmppkt: Check for packet size mismatches
Michael Niedermayer [Mon, 5 Dec 2016 10:14:51 +0000 (11:14 +0100)]
rtmppkt: Check for packet size mismatches

When receiving fragmented packets, the first packet declares the size,
and the later ones normally are small follow-on packets that don't repeat
the size and the other header fields. But technically, the later fragments
also can have a full header, declaring a different size than the previous
packet.

If the follow-on packet declares a larger size than the initial one, we
could end up writing outside of the allocation.

This fixes out of bounds writes.

Found-by: Paul Cher <paulcher@icloud.com>
Reviewed-by: Paul Cher <paulcher@icloud.com>
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit a4fec9a7eab842ea5eea1b1ee98624356cb31422)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoh264dec: fix dropped initial SEI recovery point
John Stebbins [Thu, 23 Feb 2017 21:22:56 +0000 (14:22 -0700)]
h264dec: fix dropped initial SEI recovery point

(cherry picked from commit 248dc5c1646dcdd96fe79761105c4ae889e711fd)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoasfdec: fix reading files larger than 2GB
John Stebbins [Thu, 23 Feb 2017 23:47:58 +0000 (16:47 -0700)]
asfdec: fix reading files larger than 2GB

avio_skip returns file position and overflows int

(cherry picked from commit cd7a2e1502f174c725c0de82711d2c7649057574)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoasfdec: Account for different Format Data sizes
Alexandra Hájková [Wed, 8 Feb 2017 11:51:37 +0000 (12:51 +0100)]
asfdec: Account for different Format Data sizes

Some muxers may use the BMP_HEADER Format Data size instead
of the ASF-specific one.

Bug-Id: 1020

Signed-off-by: Diego Biurrun <diego@biurrun.de>
(cherry picked from commit 0539d84d985e811e5989ef27c13f7e2dda0f9b89)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agovf_fade: Make sure to not miss the last lines of a frame
Martin Storsjö [Thu, 16 Feb 2017 10:23:20 +0000 (12:23 +0200)]
vf_fade: Make sure to not miss the last lines of a frame

When slice_h is rounded up due to chroma subsampling, there's
a risk that jobnr * slice_h exceeds frame->height.

Prior to a638e9184d63, this wasn't an issue for the last slice
of a frame, since slice_end was set to frame->height for the last
slice.

a638e9184d63 tried to fix the case where other slices than the
last one would exceed frame->height (which can happen where the
number of slices/threads is very large compared to the frame
height).

However, the fix in a638e9184d63 instead broke other cases,
where slice_h * nb_threads < frame->height. Therefore, make
sure the last slice always ends at frame->height.

CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 8f5de34c8fb18fa1416e77d2cb998773a49ddb3d)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agomov: Do not try to parse multiple stsd for the same track
Luca Barbato [Sat, 11 Feb 2017 21:44:08 +0000 (21:44 +0000)]
mov: Do not try to parse multiple stsd for the same track

Bug-Id: 1017

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit b446f0e98f85e2e931b476e52b319f1c49244660)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoaac: Validate the sbr sample rate before using the value
Luca Barbato [Sat, 11 Feb 2017 14:40:20 +0000 (15:40 +0100)]
aac: Validate the sbr sample rate before using the value

Avoid a floating point exception.

Bug-Id: 1027
(cherry picked from commit ba30b74686f0cb6c9dd465ac4820059c48bf9d08)
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agoasfdec: Use the ASF stream count when iterating
John Stebbins [Thu, 12 Jan 2017 20:36:26 +0000 (13:36 -0700)]
asfdec: Use the ASF stream count when iterating

The AVFormat stream count can be larger due external factors, such as
an id3 tag appended.

Avoid an out of bound read.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit 8e67039c6312ba520945f2c01b7b14df056d5ed1)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agomatroskadec: make sure not to leave EbmlBin in an inconsistent state
Anton Khirnov [Wed, 28 Dec 2016 12:15:14 +0000 (13:15 +0100)]
matroskadec: make sure not to leave EbmlBin in an inconsistent state

If a read fails, the current code will free the data but leave the size
non-zero. Make sure the size is zeroed in such a case.

CC: libav-stable@libav.org
Bug-Id: 1001
Found-By: Kamil Frankowicz
Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
(cherry picked from commit 9026ec8aaf5fa19cb4fb266c16f608af0d863b2b)

2 years agocmdutils: update copyright year to 2017
Sean McGovern [Sun, 1 Jan 2017 21:31:15 +0000 (16:31 -0500)]
cmdutils: update copyright year to 2017

CC: libav-stable@libav.org
(cherry picked from commit d31f46e1999fab31be46f0cbce0546a5aa49fe48)

Signed-off-by: Sean McGovern <gseanmcg@gmail.com>
2 years agohttp: Check for negative chunk sizes
Martin Storsjö [Thu, 15 Dec 2016 08:24:20 +0000 (10:24 +0200)]
http: Check for negative chunk sizes

A negative chunk size is illegal and would end up used as
length for memcpy, where it would lead to memory accesses
out of bounds.

Found-by: Paul Cher <paulcher@icloud.com>
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 131644677970a3c4a0096270ea2a5b5d437c2e63)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agomatroskadec: fix SRT subtitle duration
John Stebbins [Thu, 22 Dec 2016 17:23:30 +0000 (09:23 -0800)]
matroskadec: fix SRT subtitle duration

The codec id for SRT was changed and conditionals were not updated.

(cherry picked from commit 0982152c3fb05365597978c5d7cfeeb7ced01723)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoh264dec: make sure to only end a field if it has been started
Anton Khirnov [Sun, 18 Dec 2016 10:29:25 +0000 (11:29 +0100)]
h264dec: make sure to only end a field if it has been started

Calling ff_h264_field_end() when the per-field state is not properly
initialized leads to all kinds of undefined behaviour.

CC: libav-stable@libav.org
Bug-Id: 977 978 992
(cherry picked from commit 45286a625c6ced1f5c4c842244cbb4509429abba)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agompeg12dec: move setting first_field to mpeg_field_start()
Anton Khirnov [Sat, 17 Dec 2016 16:04:55 +0000 (17:04 +0100)]
mpeg12dec: move setting first_field to mpeg_field_start()

For field picture, the first_field is set based on its previous value.
Before this commit, first_field is set when reading the picture
coding extension. However, in corrupted files there may be multiple
picture coding extension headers, so the final value of first_field that
is actually used during decoding can be wrong. That can lead to various
undefined behaviour, like predicting from a non-existing field.

Fix this problem, by setting first_field in mpeg_field_start(), which
should be called exactly once per field.

CC: libav-stable@libav.org
Bug-ID: 999
(cherry picked from commit c2fa6bb0e8703a7a6aa10e11f9ab36094416d83f)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agompeg12dec: avoid signed overflow in bitrate calculation
Anton Khirnov [Sat, 17 Dec 2016 14:07:51 +0000 (15:07 +0100)]
mpeg12dec: avoid signed overflow in bitrate calculation

CC: libav-stable@libav.org
Bug-Id: 981
Found-By: Agostino Sarubbo
(cherry picked from commit e807491fc6a336e4becc0cbc981274a8fde18aba)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agompegvideo_parser: avoid signed overflow in bitrate calculation
Anton Khirnov [Sat, 17 Dec 2016 14:07:51 +0000 (15:07 +0100)]
mpegvideo_parser: avoid signed overflow in bitrate calculation

CC: libav-stable@libav.org
Bug-Id: 981
Found-By: Agostino Sarubbo
(cherry picked from commit 58405de0951a843765625159402870c1eea3c3b1)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agomov: fix a possible invalid read in mov_read_mac_string()
Anton Khirnov [Sat, 17 Dec 2016 13:17:20 +0000 (14:17 +0100)]
mov: fix a possible invalid read in mov_read_mac_string()

When the input string is too large, so the second condition in if ()
fails, the code will erroneously execute the else branch, indexing the
mac_to_unicode table with a negative index.

CC: libav-stable@libav.org
Bug-Id: 1000
Found-By: Kamil Frankowicz
(cherry picked from commit 46191a2da16f751e53d93646ae1388d421d12bee)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agohevcdec: do not set decoder-global SPS prematurely
Anton Khirnov [Sun, 18 Dec 2016 21:06:32 +0000 (22:06 +0100)]
hevcdec: do not set decoder-global SPS prematurely

It should only be set after the decoder state has been fully initialized
for using that SPS.
Fixes possible invalid reads on get_format() failure.

CC: libav-stable@libav.org
(cherry picked from commit 373fd76b4dbd9aa03ed28e502f33f2ca8c1ce19a)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agohevc: decouple calling get_format() from exporting the SPS parameters
Anton Khirnov [Sun, 2 Oct 2016 13:39:10 +0000 (15:39 +0200)]
hevc: decouple calling get_format() from exporting the SPS parameters

This makes sure ff_get_format() does not get called unnecessarily from
update_thread_context().

(cherry picked from commit 86157e6db2c7a9222f77fa7e7f50fb9aebc3aa81)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agoh264_slice: Wait for refs to be available before we use them in error concealment
Derek Buitenhuis [Mon, 12 Dec 2016 14:33:27 +0000 (14:33 +0000)]
h264_slice: Wait for refs to be available before we use them in error concealment

This could happen when there was a frame number gap and frame threading was used.

Debugging-by: Ronald S. Bultje <rsbultje@gmail.com>
Debugging-by: Justin Ruggles <justin.ruggles@gmail.com>
Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
CC:libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 5c7f2cf81df06614f255f061850132355a01d75e)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agomss2: only use error correction for matching block counts
Andreas Cadhalpun [Thu, 24 Nov 2016 22:57:46 +0000 (23:57 +0100)]
mss2: only use error correction for matching block counts

This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2
with coded_width/coded_height larger than width/height.

Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
(cherry picked from commit 1762a39e09a3edc27d1ef7bc50070f496b893aa4)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agoavconv: Fix the audio next dts computation
Luca Barbato [Mon, 28 Nov 2016 19:52:47 +0000 (20:52 +0100)]
avconv: Fix the audio next dts computation

Use the correct timebase.

CC: libav-stable@libav.org
(cherry picked from commit d0c84c41d33ffd270d5f9fe0290e08341397fdee)
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
2 years agomov: Update colr values
Vittorio Giovara [Tue, 18 Oct 2016 22:21:31 +0000 (18:21 -0400)]
mov: Update colr values

For 'nclx', the latest edition of the standard switched from JPEG XR
to 23001-8, which matches the current order of our entries. Bounds
are preserved as a sanity check.

For 'nclc', qtff edition 2016-09-13 introduced a few new entries.

Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
2 years agohwcontext_vaapi: Don't abort on failing to allocate from a fixed-size pool
Mark Thompson [Fri, 25 Nov 2016 12:36:05 +0000 (12:36 +0000)]
hwcontext_vaapi: Don't abort on failing to allocate from a fixed-size pool

(cherry picked from commit d30719e62de68975cbc7ffd318df03a183037563)

2 years agompegvideo: Return correct coded frame sizes from parser
Mark Thompson [Wed, 26 Oct 2016 20:12:02 +0000 (21:12 +0100)]
mpegvideo: Return correct coded frame sizes from parser

(cherry picked from commit 3297577f3eac1c87d48dedd527942de2bd28e7a5)

2 years agoavpacket: fix leak on realloc in av_packet_add_side_data()
James Almer [Sat, 19 Nov 2016 18:33:58 +0000 (15:33 -0300)]
avpacket: fix leak on realloc in av_packet_add_side_data()

If realloc fails, the pointer is overwritten and the previously allocated buffer
is leaked, which goes against the expected functionality of keeping the packet
unchanged in case of error.

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit aa498c3183236a93206b4a0e8225b9db0660b50d)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agolibopusdec: default to stereo for invalid number of channels
Andreas Cadhalpun [Mon, 14 Nov 2016 20:41:45 +0000 (21:41 +0100)]
libopusdec: default to stereo for invalid number of channels

This fixes an out-of-bounds read if avc->channels is 0.

Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit f92d7bdfddfaac04b3bb31f2749d173ca1d8ba6d)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
2 years agolibilbc: support for latest git of libilbc
Gianluigi Tiesi [Fri, 19 Sep 2014 02:49:36 +0000 (04:49 +0200)]
libilbc: support for latest git of libilbc

In the latest git commits of libilbc developers removed WebRtc_xxx typedefs.
This commit uses int types instead. It's safe to apply also for previous
versions since WebRtc_Word16 was always a typedef of int16_t and
WebRtc_UWord16 a typedef of uint16_t.

Reviewed-by: Timothy Gu <timothygu99@gmail.com>
(cherry picked from commit e17567a831dede1f24e3a1a4c305a93012d7a8ce)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
3 years agovaapi_h264: fix RefPicList[] field flags.
Gwenole Beauchesne [Wed, 19 Oct 2016 22:42:53 +0000 (23:42 +0100)]
vaapi_h264: fix RefPicList[] field flags.

Use new H264Ref.reference field to track field picture flags. The
H264Picture.reference flag in DPB is now irrelevant here.

This is a regression from git commit a12d3188, and that affected
multiple interlaced video streams.

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
(cherry picked from commit 754b20d7ebccbe8d316b12128c8cb433d5a516ac)
Signed-off-by: Mark Thompson <sw@jkqxz.net>
3 years agoPrepare for 12 Release v12
Sean McGovern [Sun, 16 Oct 2016 21:03:15 +0000 (17:03 -0400)]
Prepare for 12 Release

3 years agortmpproto: Send chunk size on the network channel
Martin Storsjö [Thu, 13 Oct 2016 13:17:11 +0000 (16:17 +0300)]
rtmpproto: Send chunk size on the network channel

This makes sure that e.g. Adobe FME actually reacts to it. As long
as the value we've been sending is the default one (128), the bug
hasn't been noticed.

Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 8b5e0d17e70400eaf5dc3845b5c1df8b2b88d830)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agortmpproto: Lengthen the filename buffer when receiving streams
Martin Storsjö [Thu, 13 Oct 2016 12:25:33 +0000 (15:25 +0300)]
rtmpproto: Lengthen the filename buffer when receiving streams

Some applications such as Adobe FME append lots of parameters
here, making it easily overflow the current limit.

Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit d6ded94036e43a04889f4ff2813a7f7dd60b82fe)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agortmpproto: Check the return from ff_amf_read_string
Martin Storsjö [Thu, 13 Oct 2016 12:24:54 +0000 (15:24 +0300)]
rtmpproto: Check the return from ff_amf_read_string

If this failed, we used to continue with an uninitialized
filename buffer.

CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit 7395784ba72742b6daa62d35db4028e09f3fdf06)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agoavformat/matroska: fix MatroskaVideoFieldOrder enum values
James Almer [Thu, 13 Oct 2016 13:53:41 +0000 (10:53 -0300)]
avformat/matroska: fix MatroskaVideoFieldOrder enum values

The spec says

9: Interlaced with bottom field displayed first and top field stored first
14: Interlaced with top field displayed first and bottom field stored first

And avcodec.h states

AV_FIELD_TB,          //< Top coded first, bottom displayed first
AV_FIELD_BT,          //< Bottom coded first, top displayed first

Signed-off-by: James Almer <jamrial@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
(cherry picked from commit 6c31ba226968f12f898120dbb928dab34e03782b)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
Conflicts:
libavformat/version.h

3 years agofile protocol: handle the file: protocol string in file_check
Anton Khirnov [Thu, 13 Oct 2016 08:03:18 +0000 (10:03 +0200)]
file protocol: handle the file: protocol string in file_check

This is consistent with what file_open() does.

CC: libav-stable@libav.org
(cherry picked from commit 20b75970e43a030f959b17ff2dfd561174b6f24e)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agoPrepare for 12_beta1 Release v12_beta1
Sean McGovern [Sun, 2 Oct 2016 20:33:03 +0000 (16:33 -0400)]
Prepare for 12_beta1 Release

3 years agoChangelog: update for the release branch
Anton Khirnov [Sun, 2 Oct 2016 19:39:52 +0000 (21:39 +0200)]
Changelog: update for the release branch

3 years agoAPIchanges: fill in the missing hash
Anton Khirnov [Sun, 2 Oct 2016 19:38:51 +0000 (21:38 +0200)]
APIchanges: fill in the missing hash

3 years agoh264dec: support broken files with mp4 extradata/annex b data
Anton Khirnov [Sat, 1 Oct 2016 08:03:05 +0000 (10:03 +0200)]
h264dec: support broken files with mp4 extradata/annex b data

Bug-Id: 966
(cherry picked from commit 5bf2454e7cb03609b3ec1a3cf4c22427fe5f8e36)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agoaudiodsp/x86: clear the high bits of the order parameter on 64bit
Anton Khirnov [Tue, 9 Aug 2016 12:17:15 +0000 (14:17 +0200)]
audiodsp/x86: clear the high bits of the order parameter on 64bit

Also change shl to add, since it can be faster on some CPUs.

CC: libav-stable@libav.org
(cherry picked from commit 75d98e30afab61542faab3c0f11880834653bd6b)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agoaudiodsp/x86: fix ff_vector_clip_int32_sse2
Anton Khirnov [Tue, 9 Aug 2016 12:17:15 +0000 (14:17 +0200)]
audiodsp/x86: fix ff_vector_clip_int32_sse2

This version, which is the only one doing two processing cycles per loop
iteration, computes the load/store indices incorrectly for the second
cycle.

CC: libav-stable@libav.org
(cherry picked from commit 1d6c76e11febb58738c9647c47079d02b5e10094)
Signed-off-by: Anton Khirnov <anton@khirnov.net>
3 years agortsp: Fix a crash with the RTSP muxer
Martin Storsjö [Thu, 15 Sep 2016 11:21:38 +0000 (14:21 +0300)]
rtsp: Fix a crash with the RTSP muxer

This was introduced in bc2a32969e.

The whole block that the statement was added to is only
relevant when used as a demuxer, but the other statements
there have had other if statements guarding them. Make
sure to only run this whole block if being used as a
demuxer.

CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
(cherry picked from commit df3795025337479a639cb3cd26c93a4e82ccd4db)
Signed-off-by: Martin Storsjö <martin@martin.st>
3 years agoPrepare for 12_alpha2 Release v12_alpha2
Sean McGovern [Tue, 30 Aug 2016 01:22:03 +0000 (21:22 -0400)]
Prepare for 12_alpha2 Release

3 years agoChangelog: Mention mov with multiple stsd
Vittorio Giovara [Thu, 25 Aug 2016 15:59:59 +0000 (11:59 -0400)]
Changelog: Mention mov with multiple stsd

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
3 years agoHigh Definition Compatible Digital (HDCD) decoder filter, using libhdcd
Burt P [Fri, 26 Aug 2016 14:12:30 +0000 (16:12 +0200)]
High Definition Compatible Digital (HDCD) decoder filter, using libhdcd

Signed-off-by: Burt P <pburt0@gmail.com>
Signed-off-by: Diego Biurrun <diego@biurrun.de>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
3 years agoavprobe: Fix memory leak
Vittorio Giovara [Thu, 21 Jul 2016 13:47:47 +0000 (15:47 +0200)]
avprobe: Fix memory leak

After init_opts() there needs to be an uninit_opts() call
to free the swscale context and other buffers.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
3 years agomov: Remove old b-frame/video delay heuristic
Derek Buitenhuis [Fri, 26 Aug 2016 13:30:07 +0000 (14:30 +0100)]
mov: Remove old b-frame/video delay heuristic

This was added before edts support existed, and is no longer
valid.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
3 years agomov: Remove ancient heuristic hack
Derek Buitenhuis [Fri, 26 Aug 2016 13:30:06 +0000 (14:30 +0100)]
mov: Remove ancient heuristic hack

This breaks files with legitimate single-entry edit lists,
and the hack, introduced in f03a081df09f9c4798a17d7e24446ed47924b11b,
has no link to any known sample in its commit message.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
3 years agolibvpx: Cast a pointer to const to squelch a warning
Diego Biurrun [Wed, 24 Aug 2016 15:00:11 +0000 (17:00 +0200)]
libvpx: Cast a pointer to const to squelch a warning

libavcodec/libvpxdec.c:100:57: warning: passing argument 3 of 'av_image_copy' from incompatible pointer type
         av_image_copy(picture->data, picture->linesize, img->planes,
libavutil/imgutils.h:116:6: note: expected 'const uint8_t **' but argument is of type 'unsigned char **'
 void av_image_copy(uint8_t *dst_data[4], int dst_linesizes[4],

3 years agovp56: Separate VP5 and VP6 dsp initialization
Diego Biurrun [Wed, 24 Aug 2016 10:30:15 +0000 (12:30 +0200)]
vp56: Separate VP5 and VP6 dsp initialization

VP5 has no arch-specific optimizations (nor will it get some in the
future), so it makes no sense to try to share dsp init code with VP6.

3 years agoprores: Change type of stride parameters to ptrdiff_t
Diego Biurrun [Wed, 24 Aug 2016 09:52:10 +0000 (11:52 +0200)]
prores: Change type of stride parameters to ptrdiff_t

This avoids SIMD-optimized functions having to sign-extend their
line size argument manually to be able to do pointer arithmetic.

Also adjust parameter names to be "linesize" everywhere.

3 years agocavs: Change type of stride parameters to ptrdiff_t
Diego Biurrun [Tue, 23 Aug 2016 22:18:41 +0000 (00:18 +0200)]
cavs: Change type of stride parameters to ptrdiff_t

ptrdiff_t is the correct type for array strides and similar.

3 years agovp8: Update some assembly comments left unchanged in bd66f073fe7286bd3c
Diego Biurrun [Wed, 24 Aug 2016 11:50:28 +0000 (13:50 +0200)]
vp8: Update some assembly comments left unchanged in bd66f073fe7286bd3c

3 years agovp8: Change type of stride parameters to ptrdiff_t
Diego Biurrun [Tue, 23 Aug 2016 21:26:35 +0000 (23:26 +0200)]
vp8: Change type of stride parameters to ptrdiff_t

ptrdiff_t is the correct type for array strides and similar.

3 years agovp56: Change type of stride parameters to ptrdiff_t
Diego Biurrun [Tue, 23 Aug 2016 21:23:45 +0000 (23:23 +0200)]
vp56: Change type of stride parameters to ptrdiff_t

This avoids SIMD-optimized functions having to sign-extend their
line size argument manually to be able to do pointer arithmetic.

3 years agovp3: Change type of stride parameters to ptrdiff_t
Diego Biurrun [Tue, 23 Aug 2016 21:19:30 +0000 (23:19 +0200)]
vp3: Change type of stride parameters to ptrdiff_t

This avoids SIMD-optimized functions having to sign-extend their
stride argument manually to be able to do pointer arithmetic.

Also adjust parameter names to be "stride" everywhere.