ovsdb-tool: Make "query" and "transact" commands work properly.
authorBen Pfaff <blp@nicira.com>
Fri, 6 Nov 2009 21:35:32 +0000 (13:35 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 6 Nov 2009 23:33:47 +0000 (15:33 -0800)
These were passing O_RDONLY or O_RDWR as arguments to a function that
expected "true" or "false", respectively.

ovsdb/ovsdb-tool.c

index 5169653d81e88f622a3b429556fc50796a5a421a..d6e0ff94120fc856db1aa4bb47cffdbe7f6a2f82 100644 (file)
@@ -162,12 +162,12 @@ do_create(int argc UNUSED, char *argv[])
 }
 
 static void
-transact(int flags, const char *db_file_name, const char *transaction)
+transact(bool read_only, const char *db_file_name, const char *transaction)
 {
     struct json *request, *result;
     struct ovsdb *db;
 
-    check_ovsdb_error(ovsdb_open(db_file_name, flags, &db));
+    check_ovsdb_error(ovsdb_open(db_file_name, read_only, &db));
 
     request = parse_json(transaction);
     result = ovsdb_execute(db, request, 0, NULL);
@@ -180,13 +180,13 @@ transact(int flags, const char *db_file_name, const char *transaction)
 static void
 do_query(int argc UNUSED, char *argv[])
 {
-    transact(O_RDONLY, argv[1], argv[2]);
+    transact(true, argv[1], argv[2]);
 }
 
 static void
 do_transact(int argc UNUSED, char *argv[])
 {
-    transact(O_RDWR, argv[1], argv[2]);
+    transact(false, argv[1], argv[2]);
 }
 
 static void