Reviewed-by: Michael Bradshaw
Reviewed-by: Paul B Mahol

Reviewed-by: Paul B Mahol

Suggested by Jean First.

fixes ticket #923
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NJavier Cabezas <jcabgz@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* qatar/master:
  avprobe, cmdutils: K&R formatting cosmetics
  tests: K&R formatting cosmetics for test programs
  lavf: free packets for muxers implementing interleave_packet().
  lavf: fix and extend av_interleaved_write_frame() doxy.
  mov: Remove dead stores for spherical coordinates for channel position.
  error_resilience: K&R formatting cosmetics
  RELEASE_NOTES: mention hiding private symbols in shared builds.
  RELEASE_NOTES: mention some notable API changes in 0.8

Conflicts:
	cmdutils.h
	doc/RELEASE_NOTES
	ffprobe.c
	libavcodec/error_resilience.c
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

This one was missed in the previous fraps fix, the
allocation is exactly the same in both cases.
Fixes fraps-v5 under valgrind.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

Fixes use of uninitialized data errors under valgrind.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

The ABI differences are negligible, but its easier for all if
all distros have libpostproc HEAD under the same soname and
debian bumped soname without consulting upstream, so as silly as
it is following this is probably the least pain for all.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes FATE failures due to uninitialized reads under valgrind
for these two codecs.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

Fixes a memleak.

Specify that lavf is responsible for freeing the data.

Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Reviewed-by: NLuca Abeni <lucabe72@email.it>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Reviewed-by: NLuca Abeni <lucabe72@email.it>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This is needed in case the get_buffer() callback doesnt set
width/height.
Ideally all decoders would make calls through some wraper
to the callbacks and that wraper would call ff_init_buffer_info()
But until thats done, the default reget buffer must call this
itself as it needs the values for the changed size check later.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* qatar/master:
  tta: cast output data pointer to the correct type
  avconv: fix -frames for video encoders with delay.
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

Second attempt at fixing ticket #921.

There is no guarantee that the casted double which is assigned to the
variable will be contained in an int (also if it is almost sure for most
non-alien architectures).

Use long long int to contain such values instead of an int, which is
required to contain at least 64 bits, so it is guaranteed to contain also
int64_t values, which are used by some fields.

In particular, should fix trac ticket #921.

Obviously the packet allocated by the encoder must be freed somewhere.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

This fixes demuxing of 2011_DCPTEST_24FPS.V.mxf.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This fixes parallel FATE (make fate -j4) failing under valgrind with:
 Syscall param ioctl(TCSET{S,SW,SF}) points to uninitialised byte(s)
    at 0x5D98B23: tcsetattr (tcsetattr.c:88)
    by 0x43D66C: term_init (ffmpeg.c:734)
    by 0x43CD8D: main (ffmpeg.c:5071)
  Address 0x7fefffdd0 is on thread 1's stack
  Uninitialised value was created by a stack allocation
    at 0x43D5B0: term_init (ffmpeg.c:716)
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

The same as av_fast_malloc but uses av_mallocz and keeps extra
always-0 padding.
This does not mean the memory will be 0-initialized after each call,
but actually only after each growth of the buffer.
However this makes sure that
a) all data anywhere in the buffer is always initialized
b) the padding is always 0
c) the user does not have to bother with adding the padding themselves
Fixes another valgrind warning about use of uninitialized data,
this time with fate-vsynth1-jpegls.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

This combination is quite odd and almost certainly a bug if
it happens.
Reviewed-by: NJustin Ruggles <justin.ruggles@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>