[FFmpeg-devel] [PATCH] add md5 muxer

Reimar Döffinger Reimar.Doeffinger
Sun Sep 27 19:53:18 CEST 2009


On Sun, Sep 27, 2009 at 07:31:00PM +0200, Michael Niedermayer wrote:
> On Sun, Sep 27, 2009 at 12:04:27PM +0200, Reimar D?ffinger wrote:
> > we have -f crc, but IMO 32 bit crc are a bit little for a whole file and
> > also we use md5sum in the regression tests, so I'd propose to add a -f
> > md5 as in attached patch.
> 
> why md5 instead of sha1?

Because md5 makes more sense for the simple reasons that
1) we already use it in regression tests
2) a md5/md5sum binary is available anywhere, I do not know
   something comparable for sha
3) for security-relevant stuff they are all "useless"

Which leaves me to ask: why sha1? I don't really see anything in
its favour (though supporting both, is trivial, the only question
would be how, I do not like any of the methods proposed so far).



More information about the ffmpeg-devel mailing list