diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2016-08-18 22:23:32 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2016-08-18 23:36:18 +0200 |
| commit | 382a68b0088b06b8df20d0133d767d53d8f161ef (patch) | |
| tree | 0dfb57b545a2496ba81ef1955c70b3f7ade2a03c | |
| parent | cc13bc8c4f0f4afa30d0b94c3f3a369ccd2aaf0b (diff) | |
| download | ffmpeg-382a68b0088b06b8df20d0133d767d53d8f161ef.tar.gz | |
vcodec/h2645_parse: Clear buffer padding
Fixes use of uninitialized memory
Fixes: 044100cb22845944988a4bd821ff8074/asan_heap-oob_329927a_1366_c3de34ce9217dac820fbb46171031bbb.jsv
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavcodec/h2645_parse.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libavcodec/h2645_parse.c b/libavcodec/h2645_parse.c index 4ed4c9a3a2..e92e38a722 100644 --- a/libavcodec/h2645_parse.c +++ b/libavcodec/h2645_parse.c @@ -34,7 +34,7 @@ int ff_h2645_extract_rbsp(const uint8_t *src, int length, { int i, si, di; uint8_t *dst; - int64_t padding = small_padding ? AV_INPUT_BUFFER_PADDING_SIZE : MAX_MBPAIR_SIZE; + int64_t padding = small_padding ? 0 : MAX_MBPAIR_SIZE; nal->skipped_bytes = 0; #define STARTCODE_TEST \ @@ -90,8 +90,8 @@ int ff_h2645_extract_rbsp(const uint8_t *src, int length, return length; } - av_fast_malloc(&nal->rbsp_buffer, &nal->rbsp_buffer_size, - length + padding); + av_fast_padded_malloc(&nal->rbsp_buffer, &nal->rbsp_buffer_size, + length + padding); if (!nal->rbsp_buffer) return AVERROR(ENOMEM); |