summaryrefslogtreecommitdiff
path: root/ovsdb/mutation.c
diff options
context:
space:
mode:
authorBen Pfaff <blp@nicira.com>2010-02-10 10:51:11 -0800
committerBen Pfaff <blp@nicira.com>2010-02-11 10:35:28 -0800
commit8b71367c38c9d2e57261fe7f30cbfc1d20b39401 (patch)
tree8421535b4a0cdf0583490be3a47d0d6e1033e461 /ovsdb/mutation.c
parentb62aeed2ab06ecb9a3b3a82dc42ebfc2703ef52f (diff)
downloadopenvswitch-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.c3
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;