Changeset 1376 for trunk/lib/protocols_transport.c

Timestamp:

10/31/08 13:51:59 (5 years ago)

Author:

perry

Message:

If we have a complete header, but truncated at the end of it, we shouldn't return NULL from
get_payload_from_* functions so we can distinguish it from a truncated header.

File:

• trunk/lib/protocols_transport.c (modified) (4 diffs)

trunk/lib/protocols_transport.c

@@ -37 +37 @@
                         
                 default:
+                        printf("unknown ethertype=%04x\n",ethertype);
                         *proto=0;
                         return NULL;
@@ -118 +119 @@
 {
         if (remaining) {
-                if (*remaining <= sizeof(libtrace_udp_t)) {
+                if (*remaining < sizeof(libtrace_udp_t)) {
                         *remaining = 0;
                         return NULL;
@@ -131 +132 @@
         unsigned int dlen = tcp->doff*4;
         if (remaining) {
-                if (*remaining <= dlen) {
+                if (*remaining < dlen) {
                         *remaining = 0;
                         return NULL;
@@ -143 +144 @@
 {
         if (remaining) {
-                if (*remaining <= sizeof(libtrace_icmp_t)) {
+                if (*remaining < sizeof(libtrace_icmp_t)) {
                         *remaining = 0;
                         return NULL;