Wireshark mailing list archives
Re: happy birthday, bug 5531!
From: Ed Beroset <beroset () mindspring com>
Date: Thu, 05 Jan 2012 20:03:26 -0500
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 crypt2. 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 Ed ___________________________________________________________________________ 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
Current thread:
- Re: happy birthday, bug 5531! Ed Beroset (Jan 05)
- Re: happy birthday, bug 5531! Joerg Mayer (Jan 14)