diff options
author | Ben Pfaff <blp@nicira.com> | 2010-02-10 10:51:11 -0800 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2010-02-11 10:35:28 -0800 |
commit | 8b71367c38c9d2e57261fe7f30cbfc1d20b39401 (patch) | |
tree | 8421535b4a0cdf0583490be3a47d0d6e1033e461 /ovsdb/mutation.c | |
parent | b62aeed2ab06ecb9a3b3a82dc42ebfc2703ef52f (diff) | |
download | openvswitch-8b71367c38c9d2e57261fe7f30cbfc1d20b39401.tar.gz |
ovsdb: Add default case to ovsdb_mutation_set_execute().
The value of 'error' is indeterminate if m->mutator is not set to a valid
value here, so we should add a default case to handle the exception. It
shouldn't happen, of course.
Found by Clang (http://clang-analyzer.llvm.org/).
Diffstat (limited to 'ovsdb/mutation.c')
-rw-r--r-- | ovsdb/mutation.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ovsdb/mutation.c b/ovsdb/mutation.c index bd6986da6..72d7d0caf 100644 --- a/ovsdb/mutation.c +++ b/ovsdb/mutation.c @@ -379,6 +379,9 @@ ovsdb_mutation_set_execute(struct ovsdb_row *row, ovsdb_datum_subtract(dst, dst_type, arg, arg_type); error = ovsdb_mutation_check_count(dst, dst_type); break; + + default: + NOT_REACHED(); } if (error) { return error; |