cvs commit: src/sys/crypto/rijndael rijndael-alg-fst.c rijndael-alg-fst.h rijndael-api-fst.c rijndael-api-fst.h rijndael.h

Sam Leffler sam at errno.com
Fri Mar 11 15:28:34 PST 2005


Hajimu UMEMOTO wrote:
> Hi,
> 
> 
>>>>>>On Fri, 11 Mar 2005 09:37:12 -0800
>>>>>>Sam Leffler <sam at errno.com> said:
> 
> 
> sam> Why did you make this change and others before I could respond?  I told 
> sam> you I agreed with your overall goal but that there were specific isuses 
> sam> we needed to resolve before you pulled the trigger.  Less than 24 hours 
> sam> after your last note to me you committed the changes.
> 
> Sorry, I was too premature.
> 
> sam> This change in particular I'm not happy with because--as I explained to 
> sam> you--I am uncertain if there are user applications that include files 
> sam> from /usr/include/crypto that may be affected.
> 
> I changed to install crypto/rijndael/rijndael.h as
> /usr/include/crypto/rijndael.h.  The difference from
> opencrypto/rijndael.h are:
> 
>   - export four internal functions.  I realized it is required for
>     gbde(8).
> 
>   - definitions of RIJNDAEL_MAXKC, RIJNDAEL_MAXKB and RIJNDAEL_MAXNR.
>     These are correspond to MAXKC, MAXKB and MAXNR in
>     opencrypto/rijndael.h.
> 
>   - rijndael_set_key() always sets up full context.  opencrypto's one
>     has a flag to set up context for encryption only.
> 
> I confirmed that only gbde(8) refers rindael in src/sys, in our tree.
> Are you mind 3rd party application?
> 
> sam> As we discussed privately there is no immediate benefit to what you are 
> sam> doing.  In fact because of these changes we are diverging from BOTH kame 
> sam> and openbsd code bases.  I'm willing to do that but only after 
> sam> understanding the consequences.
> 
> This integration will be done by moving opencrypto/* into crypto/.  I
> think there is no conflict between opencrypto/ and crypto/, now.  Can
> I go ahead?  Or, once, should I backout my changes?

See my previous note (sent moments before this arrived)--I want you to 
give me a chance properly review what you are doing.  My original import 
of the OCF had a merged crypto+opencrypto structure.  I ended up NOT 
bringing this into CVS for several reasons.  I am still not convinced 
that what you are doing is the right approach.

For now please leave things unchanged (i.e. don't backout your rijndael 
mods).

	Sam


More information about the cvs-src mailing list