Changeset 925 for trunk/lib/format_erf.c

Timestamp:

08/21/06 22:10:42 (7 years ago)

Author:

perry

Message:

trace_write_packet isn't const -- it can promote/demote packets

File:

• trunk/lib/format_erf.c (modified) (3 diffs)

trunk/lib/format_erf.c

@@ -900 +900 @@
         return 0;
 }
+
+static bool find_compatible_linktype(libtrace_packet_t *packet)
+{
+        /* Keep trying to simplify the packet until we can find 
+         * something we can do with it */
+        do {
+                char type=libtrace_to_erf_type(trace_get_link_type(packet));
+
+                /* Success */
+                if (type != (char)-1)
+                        return true;
+
+                if (!demote_packet(packet)) {
+                        trace_set_err_out(packet->trace,
+                                        TRACE_ERR_NO_CONVERSION,
+                                        "No erf type for packet (%i)",
+                                        trace_get_link_type(packet));
+                        return false;
+                }
+
+        } while(1);
+
+        return true;
+}
                 
 static int erf_write_packet(libtrace_out_t *libtrace, 
@@ -940 +964 @@
         } else {
                 dag_record_t erfhdr;
-                int type;
                 /* convert format - build up a new erf header */
                 /* Timestamp */
@@ -950 +973 @@
                         erfhdr.flags.iface = trace_get_direction(packet);
 
-                /* Keep trying to simplify the packet until we can find 
-                 * something we can do with it */
-                do {
-                        type=libtrace_to_erf_type(trace_get_link_type(packet));
-                } while(type==(char)-1 && demote_packet((libtrace_packet_t *)packet));
-                /* We just don't support this link type sorry */
-                if (type==(char)-1) {
-                        trace_set_err_out(libtrace,
-                                        TRACE_ERR_NO_CONVERSION,
-                                        "No erf type for packet");
+                if (!find_compatible_linktype(packet))
                         return -1;
-                }
-                erfhdr.type = type;
+
+                erfhdr.type = libtrace_to_erf_type(trace_get_link_type(packet));
+
                 /* Packet length (rlen includes format overhead) */
                 assert(trace_get_capture_length(packet)>0