Fix bug that could have caused infinite loop in ofp_print_actions().
authorBen Pfaff <blp@nicira.com>
Thu, 8 Jan 2009 23:27:23 +0000 (15:27 -0800)
committerBen Pfaff <blp@nicira.com>
Thu, 8 Jan 2009 23:27:23 +0000 (15:27 -0800)
Found by -Wextra noticing that len < 0 is always false.

lib/ofp-print.c

index d8d22b2b8ee06adadf8a7ed2a38acf678dd56cf0..f49aba4321e497e419f4f633f1ec919c771fbee8 100644 (file)
@@ -448,7 +448,7 @@ ofp_print_actions(struct ds *string, const struct ofp_action_header *action,
                   size_t actions_len) 
 {
     uint8_t *p = (uint8_t *)action;
-    size_t len = 0;
+    int len = 0;
 
     ds_put_cstr(string, "actions=");
     while (actions_len > 0) {