[PATCH] fix unsafe macro calls and questionable casts in libcrypt/md5.c

Bernd Schmidt bernds_cb1 at t-online.de
Thu Jun 12 14:00:03 UTC 2008


Denys Vlasenko wrote:

> Just noticed this:
[...]
> The attached patch fixes these things. Please review.

So maybe it's ancient grotty code, but presumably it works.  Since we're 
not going to do major development on it or change any of the algorithmic 
parts, it's best left alone, so as to not break the second property by 
accident.  I've seen enough "obvious cleanup" patches subtly break 
critical functionality (see also: openssl debacle) that I think it's 
just not worth it.


Bernd
-- 
This footer brought to you by insane German lawmakers.
Analog Devices GmbH      Wilhelm-Wagenfeld-Str. 6      80807 Muenchen
Sitz der Gesellschaft Muenchen, Registergericht Muenchen HRB 40368
Geschaeftsfuehrer Thomas Wessel, William A. Martin, Margaret Seif



More information about the uClibc mailing list