[PATCH] CFB 8 Bit implementation

Jussi Kivilinna jussi.kivilinna at iki.fi
Fri Jan 27 18:59:49 CET 2017


Hello,

On 24.01.2017 16:43, Mathias L. Baumann wrote:
> Hello dear Gcrypt Devs,
> 
> as a followup to Lizas request/inqueries I now implemented CFB in 8 bit mode.
> 
> Please let me know about any changes you want to see.
> 

Looks mostly ok, just few requests:
 - Change '//' C++ style comments to /* .. */
 - Add few multibyte test-vectors from [1] and [2].
 - Add changelog to commit message, see other commit for example (see [3]).
 - Send signed DCO to mailing list, and add 'Signed-off-by' to commit message (see [3]).

> I should add that my stack burning code is more or less just guessing how it should be done by imitating what I saw in the other functions.
> Please validate that I did that correct :)
> 

Yes, it's done correctly.

-Jussi

[1] http://csrc.nist.gov/groups/STM/cavp/documents/des/tdesmmt.zip
[2] http://csrc.nist.gov/groups/STM/cavp/documents/aes/aesmmt.zip
[3] https://github.com/mathias-baumann-sociomantic/libgcrypt/blob/master/doc/HACKING


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 273 bytes
Desc: OpenPGP digital signature
URL: </pipermail/attachments/20170127/b86a87fb/attachment.sig>


More information about the Gcrypt-devel mailing list