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.
if (!ovs_bridge || br_vlan < 0) {
free(ovs_bridge);
send_simple_reply(seq, ENODEV);
- return error;
+ return ENODEV;
}
}