Re: happy birthday, bug 5531!

[Joerg Mayer <jmayer () loplof de>]

On Thu, Jan 05, 2012 at 08:03:26PM -0500, Ed Beroset wrote:
> Joerg Mayer wrote:
>
> > I have a few small questions that came up during looking at the patch
> > (not all of them relevant to this patch!):
> > - why is eax.[ch] in epan instead of epan/crypt/?
> > - why do we have files named crypt/crypt-aes.c instead of crypt/aes.c?
> > - is eax.c added to CMakeLists.txt as well?
>
> For what it's worth, I've addressed these and a few other things in an  
> updated patch.  Thanks for the feedback!
>
> https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5531
>
> Comments:
> This patch differs from the previous in a couple of mostly minor ways,  
> mostly
> inspired by comment #20.  Changes are:
> 1. Moved eax.c and eax.h to crypt
> 2. fixed minor bug involving padding.  In the cases where the code  
> should have
> added exactly one pad byte, it was instead adding seventeen pad bytes.        
> 3. added eax to CMakeLists.txt
> 4. slightly simplified header canonization code
> 5. retested and refuzzed

Committed with small warning fixes (commented out some unused code/struct).

Thanks!
   Jörg
-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe