ovsdb: Fix bug in "wait" command implementation.
authorBen Pfaff <blp@nicira.com>
Thu, 2 Sep 2010 17:06:42 +0000 (10:06 -0700)
committerJustin Pettit <jpettit@nicira.com>
Tue, 5 Oct 2010 23:43:12 +0000 (16:43 -0700)
The declaration of "error" that this commit removes shadowed an outer local
declaration of "error", which caused errors detected by this code not to be
propagated up to the outer level.

Found with GCC -Wshadow.

ovsdb/execution.c

index 5b6762f..7ce9a3f 100644 (file)
@@ -629,7 +629,6 @@ ovsdb_execute_wait(struct ovsdb_execution *x, struct ovsdb_parser *parser,
         /* Parse "rows" into 'expected'. */
         ovsdb_row_hash_init(&expected, &columns);
         for (i = 0; i < rows->u.array.n; i++) {
-            struct ovsdb_error *error;
             struct ovsdb_row *row;
 
             row = ovsdb_row_create(table);