ffmpeg.git
5 years agolavc: make avpriv_flac_is_extradata_valid() private on the next bump
Anton Khirnov [Thu, 28 Aug 2014 18:42:56 +0000 (18:42 +0000)]
lavc: make avpriv_flac_is_extradata_valid() private on the next bump

5 years agooggenc: accept only STREAMINFO extradata
Anton Khirnov [Thu, 28 Aug 2014 18:51:51 +0000 (18:51 +0000)]
oggenc: accept only STREAMINFO extradata

The reasoning is the same as for
0097cbea695e534fce39958ccd103af2fbf65831.

5 years agolavc: make avpriv_flac_parse_streaminfo() private on the next bump
Anton Khirnov [Thu, 28 Aug 2014 18:42:56 +0000 (18:42 +0000)]
lavc: make avpriv_flac_parse_streaminfo() private on the next bump

5 years agolavf: stop using avpriv_flac_parse_streaminfo()
Anton Khirnov [Thu, 28 Aug 2014 10:13:25 +0000 (10:13 +0000)]
lavf: stop using avpriv_flac_parse_streaminfo()

The only parameters needed by the demuxers are the sample rate and sample
count, which can be trivially extracted manually, without resorting to
an avpriv function.

5 years agoriffenc: do not fall back on AVCodecContext.frame_size for MP3
Anton Khirnov [Sun, 24 Aug 2014 10:28:32 +0000 (10:28 +0000)]
riffenc: do not fall back on AVCodecContext.frame_size for MP3

It will not be set unless the codec context is used as the encoding
context, which is discouraged. For MP2, av_get_audio_frame_duration()
will already set the frame size properly. For MP3, set the frame size
explicitly.

5 years agolavf: use the format context strict_std_compliance instead of the codec one
Anton Khirnov [Sat, 5 Jul 2014 09:56:16 +0000 (09:56 +0000)]
lavf: use the format context strict_std_compliance instead of the codec one

5 years agolavc,lavf: switch to the new vorbis parse API
Anton Khirnov [Wed, 29 Oct 2014 15:35:26 +0000 (16:35 +0100)]
lavc,lavf: switch to the new vorbis parse API

5 years agolavc: add a public API for parsing vorbis packets.
Anton Khirnov [Wed, 29 Oct 2014 14:04:23 +0000 (15:04 +0100)]
lavc: add a public API for parsing vorbis packets.

It is required by (at least) the ogg demuxer.

Mark the current semi-public apriv API for removal.

5 years agovorbis_parser: add an AV prefix to VorbisParseContext
Anton Khirnov [Wed, 29 Oct 2014 13:33:32 +0000 (14:33 +0100)]
vorbis_parser: add an AV prefix to VorbisParseContext

This is done in preparation for making it public.

5 years agovorbis_parser: use a dedicated AVClass for logging
Anton Khirnov [Wed, 29 Oct 2014 13:18:06 +0000 (14:18 +0100)]
vorbis_parser: use a dedicated AVClass for logging

Currently, the API takes an external AVCodecContext, which is used only
for extradata and logging. This change will allow to it to work without
an AVCodecContext in the following commits.

5 years agortmpproto: Ignore errors from the getStreamLength method
Martin Storsjö [Tue, 4 Nov 2014 07:23:35 +0000 (09:23 +0200)]
rtmpproto: Ignore errors from the getStreamLength method

It is never an error if this method failed. If rt->live was
explicitly set to 0 (known to be a recorded file), print it
as a warning, otherwise print it as a debug message.

Based on a patch by Michael Niedermayer.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agohwaccel: Deinitialize hardware acceleration early enough
Rémi Denis-Courmont [Wed, 29 Oct 2014 17:07:20 +0000 (19:07 +0200)]
hwaccel: Deinitialize hardware acceleration early enough

The application will destroy the underlying hardware handles when
get_format() gets called again. Also this ensures the
deinitialization takes place if the get_format callback returns an
error.

Regression from 1c80c9d7ef809180042257200c7b5f6b81d0b0e2.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
5 years agoaacdec: add default case in channel layout
Vittorio Giovara [Thu, 30 Oct 2014 00:52:01 +0000 (00:52 +0000)]
aacdec: add default case in channel layout

Bug-Id: CID 732291

5 years agoaacsbr: change order of operation to prevent out of array read
Michael Niedermayer [Thu, 30 Oct 2014 00:52:04 +0000 (00:52 +0000)]
aacsbr: change order of operation to prevent out of array read

CC: libav-stable@libav.org
Bug-Id: CID 732250

5 years agoaacsbr: treat 1-d arrays as such
Vittorio Giovara [Thu, 30 Oct 2014 00:52:03 +0000 (00:52 +0000)]
aacsbr: treat 1-d arrays as such

CC: libav-stable@libav.org
Bug-Id: CID 1238802 / CID 1238793 / CID 1238785
        CID 1238808 / CID 1238809

5 years agocelp_filters: don't use filter lenght as loop bound
Vittorio Giovara [Thu, 30 Oct 2014 00:51:56 +0000 (00:51 +0000)]
celp_filters: don't use filter lenght as loop bound

CC: libav-stable@libav.org
Bug-Id: CID 717906 / CID 717907 / CID 717916 /
        CID 717917 / CID 717919 / CID 732259

5 years agoerror_resilience: initialize prev_* variables
Vittorio Giovara [Thu, 30 Oct 2014 00:51:57 +0000 (00:51 +0000)]
error_resilience: initialize prev_* variables

CC: libav-stable@libav.org
Bug-Id: CID 732293 / CID 732294

5 years agomotion_est: remove dead code
Vittorio Giovara [Thu, 30 Oct 2014 00:51:52 +0000 (00:51 +0000)]
motion_est: remove dead code

mb_type is initialized to 0 and never updated before the check.

Bug-Id: CID 1238782

5 years agoget_bits: remove unused assignment
Vittorio Giovara [Thu, 30 Oct 2014 00:51:51 +0000 (00:51 +0000)]
get_bits: remove unused assignment

Bug-Id: CID 1238816

5 years agomovenc: Define the flag bits using shifts instead of as decimal numbers
Martin Storsjö [Sun, 2 Nov 2014 15:01:10 +0000 (17:01 +0200)]
movenc: Define the flag bits using shifts instead of as decimal numbers

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Don't write any iso brands in ismv files
Martin Storsjö [Wed, 29 Oct 2014 20:55:23 +0000 (22:55 +0200)]
movenc: Don't write any iso brands in ismv files

We deviate slightly from the iso specs for these files.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Don't write any tfdt atom for ismv files
Martin Storsjö [Thu, 30 Oct 2014 07:35:10 +0000 (09:35 +0200)]
movenc: Don't write any tfdt atom for ismv files

The tfdt atom shouldn't be needed in those cases, we already
write tfxd atoms for ismv anyway, which is roughly equivalent.

This avoids having to declare the iso6 brand for ismv files.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Write a 0 duration in mdhd and tkhd for an empty initial moov
Martin Storsjö [Wed, 29 Oct 2014 20:51:29 +0000 (22:51 +0200)]
movenc: Write a 0 duration in mdhd and tkhd for an empty initial moov

ISO/IEC 14496-12:2012/Cor 1:2013 is explicit about how this should be
handled. All zeros doesn't mean that the full file has got a zero
duration, only that the track samples described within the initial moov
have got zero duration. An all ones duration means an indeterminate
duration.

Keep writing a duration consisting of all ones for the ISM mode -
older windows media player versions won't play a file if this is
zero. (Newer windows media player versions play either version fine.)

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Add a flag for using default-base-is-moof in tfhd atoms
Martin Storsjö [Wed, 29 Oct 2014 09:53:21 +0000 (11:53 +0200)]
movenc: Add a flag for using default-base-is-moof in tfhd atoms

Similarly to the omit_tfhd_offset flag added in e7bf085b, this
avoids writing absolute byte positions to the file, making them
more easily streamable.

This is a new feature from 14496-12:2012, so application support
isn't necessarily too widespread yet (support for it in libav was
added in 20f95f21f in July 2014).

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Signal iso6 in compatible_brands when using tfdt
Martin Storsjö [Wed, 29 Oct 2014 15:00:38 +0000 (17:00 +0200)]
movenc: Signal iso6 in compatible_brands when using tfdt

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agodoc: Document the omit_tfhd_offset flag to the mp4 muxer
Martin Storsjö [Wed, 29 Oct 2014 13:30:22 +0000 (14:30 +0100)]
doc: Document the omit_tfhd_offset flag to the mp4 muxer

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Use a local variable consistently
Martin Storsjö [Tue, 21 Oct 2014 08:36:22 +0000 (11:36 +0300)]
movenc: Use a local variable consistently

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Don't check the custom IO flag when using faststart
Martin Storsjö [Tue, 21 Oct 2014 07:44:38 +0000 (10:44 +0300)]
movenc: Don't check the custom IO flag when using faststart

The custom IO flag actually never is set for muxers, only for
demuxers, so the check was pointless (unless a user intentionally
would set the flag to signal using custom IO).

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomovenc: Don't use track_id to decide which track is the first in a moof
Martin Storsjö [Wed, 29 Oct 2014 07:20:51 +0000 (09:20 +0200)]
movenc: Don't use track_id to decide which track is the first in a moof

If one track doesn't have any samples within a moof, no traf/trun
is written for it. When the omit_tfhd_offset flag is set, none
of the tfhd atoms have any base_data_offset set, and the implicit
offset (end of previous track fragment data, or start of the moof
for the first trun) is used.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agobethsoftvid: simplify return handling
Vittorio Giovara [Wed, 29 Oct 2014 17:02:27 +0000 (17:02 +0000)]
bethsoftvid: simplify return handling

Fixes a double free in case of av_packet_new_side_data() failure.

5 years agoassdec: check av_new_packet return value
Vittorio Giovara [Fri, 24 Oct 2014 12:15:40 +0000 (13:15 +0100)]
assdec: check av_new_packet return value

CC: libav-stable@libav.org
Bug-Id: CID 703626

5 years agojack: Use av_strerror
Luca Barbato [Sun, 26 Oct 2014 13:02:45 +0000 (14:02 +0100)]
jack: Use av_strerror

5 years agofbdev: Use av_strerror
Luca Barbato [Sun, 26 Oct 2014 13:02:44 +0000 (14:02 +0100)]
fbdev: Use av_strerror

5 years agomtv: improve header check and avoid division by zero
Vittorio Giovara [Fri, 24 Oct 2014 12:15:36 +0000 (13:15 +0100)]
mtv: improve header check and avoid division by zero

CC: libav-stable@libav.org
Bug-Id: CID 732203 / CID 732204

5 years agooggenc: remove unneeded null check
Michael Niedermayer [Fri, 24 Oct 2014 12:15:37 +0000 (13:15 +0100)]
oggenc: remove unneeded null check

The code would have segfaulted before if oggstream were NULL.

CC: libav-stable@libav.org
Bug-Id: CID 732218

5 years agoconfigure: Check only for xcb
Luca Barbato [Tue, 28 Oct 2014 09:43:52 +0000 (10:43 +0100)]
configure: Check only for xcb

xcb-utils are not needed anymore.

5 years agolavd: fix building x11grab after a6674d2
Anton Khirnov [Tue, 28 Oct 2014 06:22:00 +0000 (07:22 +0100)]
lavd: fix building x11grab after a6674d2

5 years agovdpau: return MAIN instead of BASELINE for H.264 CBP
Rémi Denis-Courmont [Sun, 26 Oct 2014 19:33:58 +0000 (21:33 +0200)]
vdpau: return MAIN instead of BASELINE for H.264 CBP

This is the same as the previous change, but for applications using the
old API (such as VLC 2.2).

Signed-off-by: Anton Khirnov <anton@khirnov.net>
5 years agovdpau/h264: request MAIN rather than BASELINE VDPAU profile for CBP
Rémi Denis-Courmont [Sun, 26 Oct 2014 19:24:12 +0000 (21:24 +0200)]
vdpau/h264: request MAIN rather than BASELINE VDPAU profile for CBP

The H.264 Constrained Baseline Profile (CBP) is a subset of both the
Main Profile and the Baseline Profile. In principles, a hardware
decoder that supports either of those can decode CBP content. As it
happens, Main is supported by all VDPAU drivers, and Baseline is not.

So favor map CBP to MP for now. Hopefully in the future libvdpau will
offer an explicit choice for CBP.

This fixes bug 757.

Signed-off-by: Anton Khirnov <anton@khirnov.net>
5 years agomxfdec: reduce loop bound in mxf_read_pixel_layout()
Tomas Härdin [Mon, 27 Oct 2014 12:59:48 +0000 (13:59 +0100)]
mxfdec: reduce loop bound in mxf_read_pixel_layout()

Makes coverity less confused and code more readable.

Bug-Id: CID 732262

5 years agobethsoftvid: check return value and clean memory
Vittorio Giovara [Fri, 24 Oct 2014 12:15:34 +0000 (13:15 +0100)]
bethsoftvid: check return value and clean memory

CC: libav-stable@libav.org
Bug-Id: CID 733777

5 years agofilmstripdec: avoid integer overflow
Vittorio Giovara [Fri, 24 Oct 2014 12:15:39 +0000 (13:15 +0100)]
filmstripdec: avoid integer overflow

CC: libav-stable@libav.org
Bug-Id: CID 732246

5 years agomatroskaenc: write correct Display{Width, Height} in stereo encoding
Vittorio Giovara [Wed, 22 Oct 2014 13:36:32 +0000 (14:36 +0100)]
matroskaenc: write correct Display{Width, Height} in stereo encoding

should be the raw amount of pixels (for example 3840x1080 for full HD side by
side) and the DisplayWidth/Height in pixels should be the amount of pixels for
one plane (1920x1080 for that full HD stream)."

So, move the aspect ratio check in the mkv_write_stereo_mode() function
and always write the embl when stereo format and/or aspect ration is set.
Also add a few comments to that function.

CC: libav-stable@libav.org
Found-by: Asan Usipov <asan.usipov@gmail.com>
5 years agoxcbgrab: XCB-based screen capture
Luca Barbato [Sun, 24 Aug 2014 12:18:22 +0000 (14:18 +0200)]
xcbgrab: XCB-based screen capture

Matches the x11grab screen capture by features.

5 years agolavf: Implement ff_brktimegm using gmtime_r
Martin Storsjö [Fri, 24 Oct 2014 08:08:28 +0000 (11:08 +0300)]
lavf: Implement ff_brktimegm using gmtime_r

While a standalone implementation is nice, we already depend on
gmtime and gmtime_r in a number of places.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agodrawtext: Remove the ifdef for localtime_r
Martin Storsjö [Fri, 24 Oct 2014 07:59:34 +0000 (10:59 +0300)]
drawtext: Remove the ifdef for localtime_r

If it isn't available in the system, we've got a fallback to
the normal localtime function, so normal code can assume it is
available as long as time_internal.h is included.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agoUse gmtime_r instead of gmtime and localtime_r instead of localtime
Martin Storsjö [Fri, 24 Oct 2014 07:46:36 +0000 (10:46 +0300)]
Use gmtime_r instead of gmtime and localtime_r instead of localtime

gmtime isn't thread safe in general. In msvcrt (which lacks gmtime_r),
the buffer used by gmtime is thread specific though.

One call to localtime is left in avconv_opt.c, where thread safety
shouldn't matter (instead of making avconv depend on the libavutil
internal header).

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agolavu: Provide fallbacks for gmtime_r and localtime_r
Martin Storsjö [Fri, 24 Oct 2014 07:33:19 +0000 (10:33 +0300)]
lavu: Provide fallbacks for gmtime_r and localtime_r

This allows writing most code as if they always are is available.

These are ok to use from other libraries even though it's not a
public header, since they only provide an inline declaration, and
doesn't add an actual dependency on lavu internals. (This can be
considered more a build system compatibility fallback than a
libavutil feature.)

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agolavf: Check the return value of strftime
Martin Storsjö [Fri, 24 Oct 2014 07:43:20 +0000 (10:43 +0300)]
lavf: Check the return value of strftime

If the buffer provided to strftime is too small, the buffer contents
are indeterminate - it does not guarantee actually null terminating
the buffer.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agowtv: Avoid needlessly calling gmtime twice with the same argument
Martin Storsjö [Fri, 24 Oct 2014 07:36:54 +0000 (10:36 +0300)]
wtv: Avoid needlessly calling gmtime twice with the same argument

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agomov: fix assigment check
Vittorio Giovara [Mon, 20 Oct 2014 13:11:25 +0000 (14:11 +0100)]
mov: fix assigment check

CC: libav-stable@libav.org
Bug-Id: CID 1197050

5 years agomxfenc: Fix possible integer overflows
Tomas Härdin [Thu, 23 Oct 2014 23:05:59 +0000 (00:05 +0100)]
mxfenc: Fix possible integer overflows

None of these are likely unless the user is writing a file with two billion
streams or a duration of around two months.

CC: libav-stable@libav.org
Bug-Id: CID 700568 / CID 700569 / CID 700570 /
        CID 700571 / CID 700572 / CID 700573

5 years agomxfdec: add missing break
Vittorio Giovara [Thu, 23 Oct 2014 23:05:57 +0000 (00:05 +0100)]
mxfdec: add missing break

CC: libav-stable@libav.org
Bug-Id: CID 732232

5 years agomatroskaenc: check avio_open_dyn_buf return value
Vittorio Giovara [Thu, 23 Oct 2014 23:05:55 +0000 (00:05 +0100)]
matroskaenc: check avio_open_dyn_buf return value

CC: libav-stable@libav.org
Bug-Id: CID 703629

5 years agomatroskadec: check return values
Vittorio Giovara [Thu, 23 Oct 2014 23:05:53 +0000 (00:05 +0100)]
matroskadec: check return values

CC: libav-stable@libav.org
Bug-Id: CID 733712

5 years agomatroskadec: fix leak on error
Vittorio Giovara [Thu, 23 Oct 2014 23:05:52 +0000 (00:05 +0100)]
matroskadec: fix leak on error

CC: libav-stable@libav.org
Bug-Id: CID 1026767

5 years agolavf: replace rename() with ff_rename()
Luca Barbato [Sun, 19 Oct 2014 22:48:49 +0000 (00:48 +0200)]
lavf: replace rename() with ff_rename()

The new function wraps errno so that its value is correctly reported
when other functions overwrite it (eg. in case of logging).

CC: libav-stable@libav.org
Bug-Id: CID 1135748
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
5 years agoflac_picture: prevent a possible out of bound write
Vittorio Giovara [Mon, 20 Oct 2014 13:11:21 +0000 (14:11 +0100)]
flac_picture: prevent a possible out of bound write

At "mimetype[len] = 0;" mimetype is a 64 element array and len might be
equal to or greater than that.

CC: libav-stable@libav.org
Bug-Id: CID 1061055

5 years agoimg2dec: check av_new_packet return value
Vittorio Giovara [Mon, 20 Oct 2014 13:11:15 +0000 (14:11 +0100)]
img2dec: check av_new_packet return value

CC: libav-stable@libav.org
Bug-Id: CID 1087077

5 years agoaudiointerleave: check av_new_packet return value
Vittorio Giovara [Mon, 20 Oct 2014 13:11:14 +0000 (14:11 +0100)]
audiointerleave: check av_new_packet return value

CC: libav-stable@libav.org
Bug-Id: CID 1087078

5 years agoavfilter: check filter link validity
Vittorio Giovara [Mon, 20 Oct 2014 13:11:13 +0000 (14:11 +0100)]
avfilter: check filter link validity

Remove now redundant check.

CC: libav-stable@libav.org
Bug-Id: CID 700371

5 years agomp3enc: write full LAME frame
Anton Khirnov [Sat, 12 Apr 2014 13:20:57 +0000 (15:20 +0200)]
mp3enc: write full LAME frame

Most importantly, it contains the encoder delay and replaygain info.

5 years agoavconv: copy stream-level side data when streamcopying
Anton Khirnov [Sat, 18 Oct 2014 16:19:55 +0000 (18:19 +0200)]
avconv: copy stream-level side data when streamcopying

5 years agodoc: reword the mp3 muxer documentation
Anton Khirnov [Sat, 12 Apr 2014 13:20:01 +0000 (15:20 +0200)]
doc: reword the mp3 muxer documentation

Make it more structured.

5 years agomp3dec: fix reading the Xing tag
Anton Khirnov [Sat, 18 Oct 2014 14:25:16 +0000 (16:25 +0200)]
mp3dec: fix reading the Xing tag

The quality scale field is only supposed to be present if the fourth bit
is set. In practice, lame always sets it, but other tools might not.

CC:libav-stable@libav.org

5 years agolavf: Use av_gettime_relative
Martin Storsjö [Wed, 22 Oct 2014 09:40:46 +0000 (12:40 +0300)]
lavf: Use av_gettime_relative

The ones left using av_gettime are NTP timestamps (for RTCP,
which is specified to send the actual current realtime clock
in RTCP SR packets), and the NUT muxer timestamper, which is
documented as using wallclock time.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agoavconv: Use av_gettime_relative
Martin Storsjö [Wed, 22 Oct 2014 09:25:23 +0000 (12:25 +0300)]
avconv: Use av_gettime_relative

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agoavplay: Use av_gettime_relative
Martin Storsjö [Wed, 22 Oct 2014 09:23:08 +0000 (12:23 +0300)]
avplay: Use av_gettime_relative

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agotools: Use av_gettime_relative
Olivier Langlois [Tue, 6 May 2014 21:16:50 +0000 (17:16 -0400)]
tools: Use av_gettime_relative

Whenever av_gettime() is used to measure relative period of time,
av_gettime_relative() is prefered as it guarantee monotonic time
on supported platforms.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agolavc: Use av_gettime_relative
Olivier Langlois [Tue, 6 May 2014 21:16:49 +0000 (17:16 -0400)]
lavc: Use av_gettime_relative

Whenever av_gettime() is used to measure relative period of time,
av_gettime_relative() is prefered as it guarantee monotonic time
on supported platforms.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agolavu: Add av_gettime_relative
Martin Storsjö [Wed, 22 Oct 2014 09:14:37 +0000 (12:14 +0300)]
lavu: Add av_gettime_relative

Since av_gettime() is used in a number of places where actual
real time clock is required, the monotonic clock introduced in
ebef9f5a5 would have consequences that are hard to handle. Instead
split it into a separate function that can be used in the cases
where only relative time is desired.

On platform where no monotonic clock is available, the difference
between the two av_gettime functions is not clear, and one could
mistakenly use the relative clock where an absolute one is
required. Therefore add an offset, to make it evident that the
time returned from av_gettime_relative never is actual current
real time, even though it is based on av_gettime.

Based on a patch by Olivier Langlois.

Signed-off-by: Martin Storsjö <martin@martin.st>
5 years agonutdec: check av_new_packet return value
Vittorio Giovara [Mon, 20 Oct 2014 13:11:19 +0000 (14:11 +0100)]
nutdec: check av_new_packet return value

CC: libav-stable@libav.org
Bug-Id: CID 733713

5 years agonutenc: check for negative index rather than assert
Vittorio Giovara [Mon, 20 Oct 2014 13:11:20 +0000 (14:11 +0100)]
nutenc: check for negative index rather than assert

CC: libav-stable@libav.org
Bug-Id: CID 703721

5 years agormdec: stricter error check to avoid theoretical unitialized use
Vittorio Giovara [Mon, 20 Oct 2014 13:11:16 +0000 (14:11 +0100)]
rmdec: stricter error check to avoid theoretical unitialized use

CC: libav-stable@libav.org
Bug-Id: CID 90558

5 years agormdec: check av_new_packet return value
Vittorio Giovara [Mon, 20 Oct 2014 13:11:17 +0000 (14:11 +0100)]
rmdec: check av_new_packet return value

CC: libav-stable@libav.org
Bug-Id: CID 733714

5 years agoflvdec: make sure to check create_stream and report the same error
Vittorio Giovara [Mon, 20 Oct 2014 11:56:41 +0000 (12:56 +0100)]
flvdec: make sure to check create_stream and report the same error

CC: libav-stable@libav.org
Bug-Id: CID 732242

5 years agoflvdec: avoid unitialized use of a struct member
Vittorio Giovara [Mon, 20 Oct 2014 11:56:42 +0000 (12:56 +0100)]
flvdec: avoid unitialized use of a struct member

CC: libav-stable@libav.org
Bug-Id: CID 718141

5 years agosmoothstreamingenc: explict cast to avoid overflow
Vittorio Giovara [Mon, 20 Oct 2014 11:56:40 +0000 (12:56 +0100)]
smoothstreamingenc: explict cast to avoid overflow

CC: libav-stable@libav.org
Bug-Id: CID 732248

5 years agofate-mpeg4: use TARGET_SAMPLES for resize tests
Janne Grunau [Tue, 21 Oct 2014 07:56:23 +0000 (09:56 +0200)]
fate-mpeg4: use TARGET_SAMPLES for resize tests

5 years agortpproto: Free the addrinfo pointer on failure
Luca Barbato [Sat, 18 Oct 2014 15:58:41 +0000 (16:58 +0100)]
rtpproto: Free the addrinfo pointer on failure

CC: libav-stable@libav.org
Bug-Id: CID 1238797

5 years agonutdec: Prevent a memory corruption
Luca Barbato [Sat, 18 Oct 2014 15:58:47 +0000 (16:58 +0100)]
nutdec: Prevent a memory corruption

Chapters do not have an event_flags field.

Bug-Id: CID 1231990

5 years agortpdec_hevc: drop unnecessary check
Vittorio Giovara [Sat, 18 Oct 2014 15:58:42 +0000 (16:58 +0100)]
rtpdec_hevc: drop unnecessary check

len is always >=1 in that case.

Bug-Id: CID 1238784

5 years agortmpproto: remove dead code
Vittorio Giovara [Sat, 18 Oct 2014 15:58:44 +0000 (16:58 +0100)]
rtmpproto: remove dead code

Expression already evaluated before, redundant since
053386864219eccbcca1886c55f902f9555428a5.

Bug-Id: CID 732199

5 years agovf_format: check input validity
Vittorio Giovara [Sat, 18 Oct 2014 15:10:35 +0000 (16:10 +0100)]
vf_format: check input validity

CC: libav-stable@libav.org
5 years agowtv: clean memory on error
Vittorio Giovara [Sat, 18 Oct 2014 00:12:18 +0000 (01:12 +0100)]
wtv: clean memory on error

CC: libav-stable@libav.org
Bug-Id: CID 718002

5 years agowtv: check seek_by_sector return value
Vittorio Giovara [Sat, 18 Oct 2014 00:12:19 +0000 (01:12 +0100)]
wtv: check seek_by_sector return value

CC: libav-stable@libav.org
Bug-Id: CID 1198258

5 years agoaviobuf: check context before using it
Vittorio Giovara [Sat, 18 Oct 2014 00:12:13 +0000 (01:12 +0100)]
aviobuf: check context before using it

Avoid a possible null pointer dereference.

CC: libav-stable@libav.org
Bug-Id: CID 1135769

5 years agoavio: fix sizeof argument
Michael Niedermayer [Sat, 18 Oct 2014 00:12:12 +0000 (01:12 +0100)]
avio: fix sizeof argument

CC: libav-stable@libav.org
Bug-Id: CID 732284

5 years agoidcin: fix return check
Vittorio Giovara [Sat, 18 Oct 2014 00:12:11 +0000 (01:12 +0100)]
idcin: fix return check

CC: libav-stable@libav.org
Bug-Id: CID 732198

5 years agortmp: Always call rtmp_close() on rtmp_open() failure
Alexander Drozdov [Sat, 18 Oct 2014 14:02:32 +0000 (16:02 +0200)]
rtmp: Always call rtmp_close() on rtmp_open() failure

Prevent possible memory leaks.

Connect to nginx and request a non-existent resource to
trigger the issue.

CC: libav-stable@libav.org
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Signed-off-by: Uwe L. Korn <uwelk@xhochy.com>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
5 years agoswscale: fix sign extensions in yuv planar conversion
Vittorio Giovara [Fri, 17 Oct 2014 13:31:35 +0000 (14:31 +0100)]
swscale: fix sign extensions in yuv planar conversion

Casting the left-most byte to unsigned avoids an undefined
result of the shift by 24 if bit 7 is set.

yuvPlanartouyvy_c and yuvPlanartoyuy2_c are affected.

CC: libav-stable@libav.org
Bug-Id: CID 732281 / CID 732282

5 years agooss_audio: use a macro to simplify ioctl() error checking
Timothy Gu [Wed, 15 Oct 2014 16:32:54 +0000 (17:32 +0100)]
oss_audio: use a macro to simplify ioctl() error checking

Also add a note about SNDCTL_DSP_GETFMTS which may fail even if OSS is
available.

CC: libav-stable@libav.org
Bug-Id: CID 1238992
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
5 years agoavresample: Make sure the even check does not overflow
Luca Barbato [Wed, 15 Oct 2014 16:32:57 +0000 (17:32 +0100)]
avresample: Make sure the even check does not overflow

CC: libav-stable@libav.org
Bug-Id: CID 732225

5 years agoavresample: prevent theoretical division by zero
Vittorio Giovara [Wed, 15 Oct 2014 16:33:00 +0000 (17:33 +0100)]
avresample: prevent theoretical division by zero

CC: libav-stable@libav.org
Bug-Id: CID 1231986

5 years agoaf_resample: check av_opt_set_dict return value
Vittorio Giovara [Fri, 17 Oct 2014 09:07:11 +0000 (10:07 +0100)]
af_resample: check av_opt_set_dict return value

CC: libav-stable@libav.org
Bug-Id: CID 1087076

5 years agovf_showinfo: Forward the av_image_get_linesize error
Vittorio Giovara [Fri, 17 Oct 2014 09:07:09 +0000 (10:07 +0100)]
vf_showinfo: Forward the av_image_get_linesize error

CC: libav-stable@libav.org
Bug-Id: CID 1087086

5 years agovf_drawtext: Do not leak the mmapped textfile
Luca Barbato [Fri, 17 Oct 2014 09:07:10 +0000 (10:07 +0100)]
vf_drawtext: Do not leak the mmapped textfile

And validate its size while at it.

CC: libav-stable@libav.org
Bug-Id: CID 1244189

5 years agolavc: make lmax/lmin into private options of mpegvideo encoders
Anton Khirnov [Sun, 27 Oct 2013 12:51:16 +0000 (13:51 +0100)]
lavc: make lmax/lmin into private options of mpegvideo encoders

5 years agolavc: make border_masking into private options of mpegvideo encoders
Anton Khirnov [Sun, 27 Oct 2013 12:51:16 +0000 (13:51 +0100)]
lavc: make border_masking into private options of mpegvideo encoders

5 years agolavc: deprecate unused mb_threshold field
Anton Khirnov [Wed, 8 Oct 2014 10:58:43 +0000 (10:58 +0000)]
lavc: deprecate unused mb_threshold field