ovsdb: Fix segfault in ovsdb_file_txn_row_from_json().

If 'error' is nonnull then we destroy the row, so we must not try to reuse
the row immediately after that.

Support request #6155.
Repoted-by: Geoff White <gwhite@nicira.com>

diff --git a/ovsdb/file.c b/ovsdb/file.c
index 9eea460d4dd993cc15fff3e30ffc9d03c2fd6444..05a4067a5759382276ee0f28d5ee5b83d799912e 100644 (file)
--- a/ovsdb/file.c
+++ b/ovsdb/file.c
@@ -333,10 +333,9 @@ ovsdb_file_txn_row_from_json(struct ovsdb_txn *txn, struct ovsdb_table *table,
         error = ovsdb_file_update_row_from_json(new, converting, json);
         if (error) {
             ovsdb_row_destroy(new);
+        } else {
+            ovsdb_txn_row_insert(txn, new);
         }
-
-        ovsdb_txn_row_insert(txn, new);
-
         return error;
     }
 }