Messages in this thread | | | Date | Wed, 5 Nov 2008 22:27:48 +0100 | From | Sebastian Andrzej Siewior <> | Subject | Re: [PATCH] crypto: add test vectors for skein256/512/1024 |
| |
* Jeff Garzik | 2008-11-02 20:44:40 [-0500]:
> Sebastian Andrzej Siewior wrote: >> I grabed them from http://www.schneier.com/skein.html. The last test >> vector >> (3) in every category is currently deactivated because it failed always. >> It is unlikely that I made a type because I copy+pasted the tables + vim >> magiced them. So maybe code may missbehave on requests lengths which are >> not a multiple of 4 or the vectors can not be used due to some other >> limitations that I've overseen.
Okay, that was a false positive :) The "untouched" reference implementation outputs the same hash. The additional information (tree, ...) is part of the third argument in Skein_XXX_InitExt() which isn't used. So we can safely drop the third test vector.
> > Jeff
Sebastian
| |