diff options
| author | Ben Pfaff <blp@nicira.com> | 2011-02-23 13:05:25 -0800 |
|---|---|---|
| committer | Ben Pfaff <blp@nicira.com> | 2011-02-23 15:14:42 -0800 |
| commit | 1611cf3f5abbff87b4443d2807b369fdd327eb49 (patch) | |
| tree | 9396d1008ab5b8dff247fdc71914cb9b2b9fe0e7 /utilities | |
| parent | fbca1e20e84bc34537f0a3db073195f6783fb373 (diff) | |
| download | openvswitch-1611cf3f5abbff87b4443d2807b369fdd327eb49.tar.gz | |
ovs-vsctl: Clarify that uuid_from_string() shouldn't fail in post_create().
Coverity #10701.
Diffstat (limited to 'utilities')
| -rw-r--r-- | utilities/ovs-vsctl.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c index 3a14f7136..4435f41d1 100644 --- a/utilities/ovs-vsctl.c +++ b/utilities/ovs-vsctl.c @@ -3127,7 +3127,9 @@ post_create(struct vsctl_context *ctx) const struct uuid *real; struct uuid dummy; - uuid_from_string(&dummy, ds_cstr(&ctx->output)); + if (!uuid_from_string(&dummy, ds_cstr(&ctx->output))) { + NOT_REACHED(); + } real = ovsdb_idl_txn_get_insert_uuid(ctx->txn, &dummy); if (real) { ds_clear(&ctx->output); |