Commits · 82e266c6d3fbf3cc74e515b883e66543381a0f2c · Linshizhi / ffmpeg.wasm-core

19 Sep, 2013 16 commits

segafilm: Validate the number of audio channels · 82e266c6

Martin Storsjö authored Sep 19, 2013

This avoids divisions by zero later.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

82e266c6

svq3: Avoid a division by zero · 601c2015

Martin Storsjö authored Sep 19, 2013

If the height is zero, the decompression will probably end up
failing due to not fitting into the allocated buffer later
anyway, so this doesn't need any more elaborate check.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

601c2015

qpeg: Add checks for running out of rows in qpeg_decode_inter · 7a5a5572

Martin Storsjö authored Sep 19, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

7a5a5572

mpegaudiodec: Validate that the number of channels fits at the given offset · e9d61de9

Martin Storsjö authored Sep 19, 2013

This is similar to the fix in 35cbc98b.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

e9d61de9

asvdec: Verify the amount of extradata · f5080335

Martin Storsjö authored Sep 19, 2013

The init function reads one byte of extradata.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

f5080335

movenc: Use null buffers for measuring the amount of data to be written · 72fe16a1
Martin Storsjö authored Sep 18, 2013
```
Signed-off-by: Martin Storsjö <martin@martin.st>
```
72fe16a1

aviobuf: Add functions for null buffers · 3627ce2f

Martin Storsjö authored Sep 18, 2013

Null buffers are useful for simulating writing to a real buffer
for the sake of measuring how many bytes are written.
Signed-off-by: Martin Storsjö <martin@martin.st>

3627ce2f

asfenc: mux chapters in ASF files using an ASF "marker" section · bb461370

Vladimir Pantelic authored Sep 19, 2013

ASF markers only have a start time, so we lose the chapter end times,
but that is ASF for you
Signed-off-by: Vladimir Pantelic <vladoman@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

bb461370

asfenc: remember send time and offset of the index entries · 09f3c937
Vladimir Pantelic authored Sep 19, 2013
```
Signed-off-by: Vladimir Pantelic <vladoman@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
```
09f3c937
asfenc: add ASF_Reserved_4 as defined in section 10.10 of the ASF spec · 1eb93280
Vladimir Pantelic authored Sep 19, 2013
```
Signed-off-by: Vladimir Pantelic <vladoman@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
```
1eb93280
asfdec: replace magic constant with DATA_HEADER_SIZE · c53b5dda
Vladimir Pantelic authored Sep 19, 2013
```
Signed-off-by: Vladimir Pantelic <vladoman@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>
```
c53b5dda

asfdec: substract preroll time from marker presentation time · 5f408333

Vladimir Pantelic authored Sep 19, 2013

this was forgotten when we changed ASF to not output the preroll time
Signed-off-by: Vladimir Pantelic <vladoman@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

5f408333

rv34: Fix a memory leak on errors · fdbd924b
Martin Storsjö authored Sep 16, 2013
```
Signed-off-by: Martin Storsjö <martin@martin.st>
```
fdbd924b

idroqdec: Make sure a video stream has been allocated before returning packets · bcbe4f3c

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

bcbe4f3c

rv10: Validate the dimensions set from the container · 5372cda6

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

5372cda6

rtmp: Unbreak handle_notify · ae0f316a
Luca Barbato authored Sep 18, 2013
```
Commit 5626f994 broke it.
```
ae0f316a

18 Sep, 2013 6 commits
- aacdec: Add support for LD (Low Delay) AAC · 1914e6f0
  Alex Converse authored Sep 16, 2013
  
  1914e6f0
- aacdec: Add support for Error Resilience syntax. · 5cdefc02
  Alex Converse authored Sep 16, 2013
```
This does not add support for any error resilience tools.
```
  5cdefc02
- Add a WebP decoder · c4bfa098
  Justin Ruggles authored Apr 16, 2013
```
Container and lossy decoding by Aneesh Dogra <aneesh@sugarlabs.org>
Lossless decoding by Justin Ruggles <justin.ruggles@gmail.com>
```
  c4bfa098
- lavc: add ff_u8_to_s8() for safe type punning of uint8_t to int8_t · 9ae53c58
  Justin Ruggles authored Sep 16, 2013
  
  9ae53c58
- avformat: Use av_reallocp() where suitable · 5626f994
  Alexandra Khirnova authored Sep 18, 2013
```
Signed-off-by: Diego Biurrun <diego@biurrun.de>
```
  5626f994
- rmdec: Validate the fps value · 0f310a6f
  Martin Storsjö authored Sep 16, 2013
```
Abort if it is invalid if strict error checking has been requested.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>
```
  0f310a6f
17 Sep, 2013 18 commits

movenc: Add an option for omitting the tfhd base offset · e7bf085b

Martin Storsjö authored Sep 09, 2013

This makes the output fragments independent of their position in
the output stream, making the output work better when streamed.

QuickTime Player doesn't support fragmented mp4 without the base
data offset, though.
Signed-off-by: Martin Storsjö <martin@martin.st>

e7bf085b

movenc: Write the moof atom in two passes · a4adabb4

Martin Storsjö authored Sep 09, 2013

This is a bit more work, but avoids having to fill in
the data offset field afterwards instead of directly when
the rest of the trun atom is written.

This simplifies future cases where this field needs to be set to
something different.
Signed-off-by: Martin Storsjö <martin@martin.st>

a4adabb4

twinvqdec: Check the ibps parameter separately · c77d409b

Martin Storsjö authored Sep 17, 2013

This is required, since invalid parameters actually could
pass the switch check below.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

c77d409b

rtmp: Support reading interleaved chunks. · 8583b142

Josh Allmann authored Sep 17, 2013

A given packet won't always come in contiguously; sometimes
they may be broken up on chunk boundaries by packets of another
channel.

This support primarily involves tracking information about the
data that's been read, so the reader can pick up where it left
off for a given channel.

As a side effect, we no longer over-report the bytes read if
(toread = MIN(size, chunk_size)) == size
Signed-off-by: Martin Storsjö <martin@martin.st>

8583b142

xmv: Add more sanity checks for parameters read from the bitstream · d4c2a374

Martin Storsjö authored Sep 16, 2013

Since the number of channels is multiplied by 36 and assigned to
to a uint16_t, make sure this calculation didn't overflow. (In
certain cases the calculation could overflow leaving the
truncated block_align at 0, leading to divisions by zero later.)

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

d4c2a374

rtmpproto: Extend a comment to explain the prev_pkt arrays roles · dc4acc82
Martin Storsjö authored Sep 17, 2013
```
Signed-off-by: Martin Storsjö <martin@martin.st>
```
dc4acc82

rtmpproto: Consistently use the right prev_pkt array · 647d655d

Martin Storsjö authored Sep 17, 2013

prev_pkt[0] is used for input packets, while prev_pkt[1] is
used for output.
Signed-off-by: Martin Storsjö <martin@martin.st>

647d655d

rtmpproto: Check for the right return code · 0d378439
Martin Storsjö authored Sep 17, 2013
```
Signed-off-by: Martin Storsjö <martin@martin.st>
```
0d378439

dca_parser: Handle changes in DCA frame size · 8411baf6

John Stebbins authored Sep 04, 2013

A change in framesize caused a perpetual loss of synchronization.
So read (and use) the frame size from the frame header instead of
setting it only once.
Signed-off-by: Diego Biurrun <diego@biurrun.de>

8411baf6

jpeg2000dec: Add more checks when parsing headers · 18a245a2

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

18a245a2

doc: Add missing hashes and dates to APIChanges · c38854c3
Luca Barbato authored Sep 17, 2013
```
Also fix the typo in the last entry version.
```
c38854c3

ffv1: Make sure at least one slice context is initialized · b1db3315

Martin Storsjö authored Sep 16, 2013

This avoids crashes when initializing the range coder for
the first slice context.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

b1db3315

eacmv: Make sure a reference frame exists before referencing it · ea78a348

Martin Storsjö authored Sep 16, 2013

This is similar to an existing check for the second-last frame
from 062421e3.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

ea78a348

truemotion2: Use av_freep properly in an error path · c39f7eba

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

c39f7eba

mpeg4videodec: Check the width/height in mpeg4_decode_sprite_trajectory · f875a732

Martin Storsjö authored Sep 16, 2013

This avoids a potential division by zero.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

f875a732

ivi_common: Make sure color planes have been initialized · a92538b7

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

a92538b7

mov: Don't use a negative duration for setting other fields · 7f8d41eb

Martin Storsjö authored Sep 16, 2013

Some files have the duration set to -1 in the mdhd atom, more
or less legitimately. (We produce such files ourselves, for the
initial duration in fragmented mp4 files.)

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

7f8d41eb

oggparseogm: Convert to use bytestream2 · 19b9659f

Martin Storsjö authored Sep 16, 2013

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: Martin Storsjö <martin@martin.st>

19b9659f