ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 1123] Index multiple instances of the same field name to r

Date: Mon, 30 May 2011 19:50:02 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1123

Michael Mann <mmann78@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6429|                            |review_for_checkin?
               Flag|                            |

--- Comment #15 from Michael Mann <mmann78@xxxxxxxxxxxx> 2011-05-30 19:49:58 PDT ---
Created an attachment (id=6429)
 --> (https://bugs.wireshark.org/bugzilla/attachment.cgi?id=6429)
UATification of ESP

UATification of ESP SA keys.  Some notes:
1. Trimmed a lot of fat removing the code that parsed (and reparsed) ESP SA
keys, since UATs can take care of all that. 
2. There was no validation on the src/dest, so that was carried through (ie
there is still no validation of src/dest fields)
3. Formatting of the source was a little off, so I attempted to fix the
functions I needed to deal with.  I know that's frowned upon, but the braces
were such that I couldn't tell if/else blocks to understand the code.  If
somebody wanted to run an indent program on the file, that would be
appreciated.
4. I could use some assistance in testing the changes as I have a limited
understanding of ESP (but the UAT stuff is fairly straightforward)

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.