On Thu, Dec 29, 2011 at 02:25:43PM -0500, Ed Beroset wrote: > Chris Maynard wrote: >> Ed Beroset<beroset@...> writes: >> >>> https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5531 >>> >>> It's been a year since it was originally submitted. As always, if there's >> anything I can do to help get this >>> into the main code, please let me know. I know a number of people that are >> waiting for it. And thanks again for >>> a mighty handy tool! >>> >>> Ed >> >> I know it can be frustrating when waiting for something so thanks for your >> continued patience. If it makes you feel any better, some bugs are over 6 >> years >> old. :) > > Yes, it's a bit frustrating, but I also certainly understand. I wish I > had more time to spend on this, too. I have a half-finished > documentation section on how to write ASN.1 based dissectors that I'm > hoping to finish within the next few weeks and I've been looking over > Bill's rewritten tvb_ stuff to see if I can help explain that, too. > First I'd have to understand it...
I looked at this patch a bit but as I don't know anything about BER I can't comment on much. 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? - is this in any way related to RFC 6142? - I don't know anything about BER encoding, but is the existence of the function get_ber_len_size owed to missing infrastructure in Wireshark? Thanks Jörg -- Joerg Mayer <[email protected]> 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 <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
