[FFmpeg-devel] [PATCH] avcodec/jpeg2000dec: Fix mixed declaration and code

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Apr 9 19:35:14 EEST 2020


Carl Eugen Hoyos:
> Am Do., 9. Apr. 2020 um 14:39 Uhr schrieb Andreas Rheinhardt
> <andreas.rheinhardt at gmail.com>:
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
>> ---
>>  libavcodec/jpeg2000dec.c | 5 +++--
>>  1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c
>> index 846e31d0a2..9684e57b34 100644
>> --- a/libavcodec/jpeg2000dec.c
>> +++ b/libavcodec/jpeg2000dec.c
>> @@ -862,6 +862,7 @@ static int get_plt(Jpeg2000DecoderContext *s, int n)
>>  static int get_ppt(Jpeg2000DecoderContext *s, int n)
>>  {
>>      Jpeg2000Tile *tile;
>> +    void *new;
>>
>>      if (n < 3) {
>>          av_log(s->avctx, AV_LOG_ERROR, "Invalid length for PPT data.\n");
>> @@ -879,8 +880,8 @@ static int get_ppt(Jpeg2000DecoderContext *s, int n)
>>
>>      tile->has_ppt = 1;  // this tile has a ppt marker
>>      bytestream2_get_byte(&s->g); // Zppt is skipped and not used
>> -    void* new = av_realloc(tile->packed_headers,
>> -                           tile->packed_headers_size + n - 3);
>> +    new = av_realloc(tile->packed_headers,
>> +                     tile->packed_headers_size + n - 3);
> 
> Please push such fixes directly, there is too much traffic already
> on this mailing list.
> 
Ok. Should patches like the ones contained in the patchset [1] also be
pushed directly?

- Andreas

[1]: https://ffmpeg.org/pipermail/ffmpeg-devel/2019-October/252107.html


More information about the ffmpeg-devel mailing list