[FFmpeg-devel] [PATCH 3/3] avformat/utils: Set stream side-data size even without side-data

[Anton Khirnov]

Quoting Andreas Rheinhardt (2020-05-22 03:24:31)
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
> Was the earlier behaviour actually a bug (I think so) or is this new
> behaviour something that should be explicitly documented?
> 
> Btw, the stream and packet side-data APIs differ in two more ways:
> av_packet_new_side_data() adds padding to the buffer and zeroes it
> whereas av_stream_new_side_data() does not. Should this be unified?

Given that the side data can be extradata, I'd say yes to padding.

For zeroing, not sure. The caller should fill the data completely, so
zeroing it may hide missed initialization.

-- 
Anton Khirnov