ovsdb: Improve error message for duplicate uuid-name.
authorBen Pfaff <blp@nicira.com>
Wed, 9 Mar 2011 20:42:46 +0000 (12:42 -0800)
committerBen Pfaff <blp@nicira.com>
Thu, 10 Mar 2011 19:23:59 +0000 (11:23 -0800)
commit19b48a816b9af445ff71269ba7219b64f763ca18
treeefd7af06f7200c567920e4dc570b5c8f6d8f38a8
parent1dd5b71d14b5c5360f97071041ec1bdd1f8d2445
ovsdb: Improve error message for duplicate uuid-name.

ovsdb_execute_insert() tried to return a helpful error message when there
was a duplicate uuid-name, but ovsdb_execute() (its caller) makes any
parse error override a parse error.  Since ovsdb_execute_insert() would
skip parsing the row when the uuid-name was a duplicate, this meant that
the error actually reported would be that "row" was not allowed here, which
wasn't at all helpful (since "row" is in fact mandatory).

This commit clears up the problem by always retrieving the "row" member,
which required a small amount of refactoring, and adds a test.
ovsdb/execution.c
tests/ovsdb-execution.at