[FFmpeg-devel] postprocess.c: filter name needs to be double 0 terminated

Michael Niedermayer michaelni at gmx.at
Tue May 31 17:51:30 CEST 2011


On Mon, May 30, 2011 at 01:19:35PM +0200, Piotr Kaczuba wrote:
> W dniu 2011-05-29 00:15, Michael Niedermayer pisze:
> [...]
>> not pushed, it failed tests, the code needs more bytes to be 0 than 2
>> when multiple expansions happen
>> i reverted the problematic hunk that caused all this locally
> [...]
>
> No, two 0 bytes are enough, I just forgot to take care of it after a  
> replacement happens, hence the failure. Please see attached patch, which  
> should work in all cases. But instead of ensuring that the string is  
> always double 0-terminated, I used memset() at the beginning to avoid  
> adding extra instructions in the loop.
>
> -- 
> Cheers,
> Piotr

>  postprocess.c |    7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 364dcf8335974064f5c053d7b827f6417d3eae13  postprocess.new.diff

ok, applied, thx


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20110531/10701538/attachment.asc>


More information about the ffmpeg-devel mailing list