tcpdump mailing list archives
Re: [PATCH 2/2] print-olsr: Add basic IPv6 support.
From: Guy Harris <guy () alum mit edu>
Date: Thu, 21 May 2009 10:42:45 -0700
On May 16, 2009, at 3:05 PM, Florian Forster wrote:
From: Florian Forster <octo () leeloo lan home verplant org> Unfortunately OLSR uses the same IDs for IPv4 and IPv6 packets, eventhough the size of "messages" differ. The version of the internet protocolis therefore handed to the "olsr_print" function. The code isn't very nice, due to a high density of #ifdef INET6'es. If IPv6-support really should be optional, I'm afraid this is inevitable. Both, compilation with and without IPv6 support has been tested.The patch addresses fixes other issues, too. The length given in the packetwas used for pointer arithmetic without checking if the value was in a reasonable range first in several places. It should now be possible to decode more than one "namespace message" within a single packet. Otherchanges remove trailing whitespace or fix lines indented with tabs (themajority of the file is indented using spaces).
Checked in and pushed. - This is the tcpdump-workers list. Visit https://cod.sandelman.ca/ to unsubscribe.
Current thread:
- [PATCH] print-olsr: Don't trust the package payload. Florian Forster (May 16)
- [PATCH 1/2] util.c: Add the `mask62plen' utility function. Florian Forster (May 16)
- [PATCH 2/2] print-olsr: Add basic IPv6 support. Florian Forster (May 16)
- Re: [PATCH 2/2] print-olsr: Add basic IPv6 support. Guy Harris (May 21)
- Re: [PATCH 1/2] util.c: Add the `mask62plen' utility function. Guy Harris (May 21)
- [PATCH 2/2] print-olsr: Add basic IPv6 support. Florian Forster (May 16)
- Re: [PATCH] print-olsr: Don't trust the package payload. Guy Harris (May 21)
- [PATCH 1/2] util.c: Add the `mask62plen' utility function. Florian Forster (May 16)