In C, the || operator yields 0 or 1, not (as in some other languages) the
value of its first nonzero operand.
Found by inspection.
Signed-off-by: Ben Pfaff <blp@nicira.com>
error = ofp_mkerr(OFPET_BAD_ACTION, OFPBAC_BAD_ARGUMENT);
}
- if (subtype == NXAST_BUNDLE_LOAD) {
- error = nxm_dst_check(nab->dst, nab->ofs_nbits, 16, flow) || error;
+ if (subtype == NXAST_BUNDLE_LOAD && !error) {
+ error = nxm_dst_check(nab->dst, nab->ofs_nbits, 16, flow);
}
if (slaves_size < n_slaves * sizeof(ovs_be16)) {