Hello, On Fri, Sep 01, 2006 at 01:52:25PM +1000, Herbert Xu wrote: > On Thu, Aug 31, 2006 at 02:39:30PM +0200, Rik Snel wrote: > > This is an attempt at implementing LRW for the new blockcipher API. > > Please review, test and comment. > > Thanks a lot for doing this. It looks good to me.
That's good to hear. > There are a few style (see Documentation/CodingStyle) issues. It would > be good if you can fix them up before I apply the patches. Can you give some examples about what I missed from CodingStyle? (I just reread it and I think my patch adheres pretty well to those rules: tabs=8, K&R bracing, don't pollute global namespace etc...) > Could you also convert the test vectors to use tcrypt.h/tcrypt.c? Yes, I just didn't know what tcrypt.[ch] was for, otherwise I would have put those vectors there in the first place. The last testvector, however, won't fit because it is an entire 512 byte sector. (which is not that bad, because alle narrow blocks (16 bytes) are encrypted independently, no realistic testvector of a wide block cipher mode would fit. (ABL uses blocks of any size, say 512 bytes, and if 1 bit changes in the plaintext block, the whole cypher block will be affected) I will propose something to fix that if ABL is finished. Greetings, Rik. -- Nothing is ever a total loss; it can always serve as a bad example. - To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
