diff options
author | Ben Pfaff <blp@nicira.com> | 2009-07-30 13:55:24 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2009-08-07 15:05:47 -0700 |
commit | 621fd03a0717358fb8600a8bacb99782ab181005 (patch) | |
tree | dbdfd8e62dcb52bad12fe7827cd69d10941ff2b3 /vswitchd/ovs-brcompatd.c | |
parent | 41e754bcb40f78f5a2a790a4b196433cd1afa304 (diff) |
brcompatd: Fix handle_fdb_query_cmd() return value on error.
handle_fdb_query_cmd() should return an error when an error occurs, but
in this case it was always returning 0, because error is known to have
value 0 at this point.
Nothing really uses the return value here, and so eventually it would make
sense to just change the return type here and in the rest of the
handle_*() functions to void.
Diffstat (limited to 'vswitchd/ovs-brcompatd.c')
-rw-r--r-- | vswitchd/ovs-brcompatd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vswitchd/ovs-brcompatd.c b/vswitchd/ovs-brcompatd.c index e569c74d..7f050992 100644 --- a/vswitchd/ovs-brcompatd.c +++ b/vswitchd/ovs-brcompatd.c @@ -631,7 +631,7 @@ handle_fdb_query_cmd(struct ofpbuf *buffer) if (!ovs_bridge || br_vlan < 0) { free(ovs_bridge); send_simple_reply(seq, ENODEV); - return error; + return ENODEV; } } |