On 08/01/13 15:35, Anton Khirnov wrote:
> Freeing it in av_destruct_packet(), as is done currently, would mean
> that we allow it to be allocated with other means. But that would make
> av_packet_new_side_data() unsafe.
> 
> Side data is not expected to be large, so copying it if required
> shouldn't be a problem.

The assumption is currently true so the patch is fine, we'll need to
revisit it if we want to use side data for something more.

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to