[FFmpeg-devel] [PATCH] wtvdec: *_to_iso8601: Behave less surprising

Peter Ross pross at xvid.org
Sun Jul 1 04:46:21 CEST 2012


On Sun, Jul 01, 2012 at 02:55:57AM +0200, Alexander Strasser wrote:
>   Do not violate the (implicit) contract that is indicated by
> the arguments of those functions. In other words if buf_size
> is passed as zero do not try to write to the buffer's first
> element.

wtvdec always calls the foo_to_is8601 functions with buf_size > 0.

an alternate patch would be to document that expectation (attached).

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-wtvdec-clarify-_to_iso8601-buf_size-parameter.patch
Type: text/x-diff
Size: 987 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120701/ee7401b3/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120701/ee7401b3/attachment.asc>


More information about the ffmpeg-devel mailing list