Re: [FFmpeg-devel] [PATCH 1/2] avcodec/cbs_h264: Automatically free SEI payload on error

2019-09-24 Thread Mark Thompson
On 11/09/2019 23:18, Andreas Rheinhardt wrote: > If adding an SEI message to an access unit fails, said SEI message was > not touched, so that the caller had to free any data associated with it > that might need to be freed. But given that ff_cbs_h264_add_sei_message > can simply call

[FFmpeg-devel] [PATCH 1/2] avcodec/cbs_h264: Automatically free SEI payload on error

2019-09-11 Thread Andreas Rheinhardt
If adding an SEI message to an access unit fails, said SEI message was not touched, so that the caller had to free any data associated with it that might need to be freed. But given that ff_cbs_h264_add_sei_message can simply call cbs_h264_free_sei_payload, one can easily free the content of the