The function bond_member_lookup() could return NULL, the return value
need to be checked.
Addresses-Coverity: ("Dereference null return value")
Signed-off-by: Yunjian Wang <[email protected]>
---
ofproto/bond.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/ofproto/bond.c b/ofproto/bond.c
index a4116588f..2dcfeda71 100644
--- a/ofproto/bond.c
+++ b/ofproto/bond.c
@@ -672,8 +672,13 @@ out:
void
bond_member_set_may_enable(struct bond *bond, void *member_, bool may_enable)
{
+ struct bond_member *member;
+
ovs_rwlock_wrlock(&rwlock);
- bond_member_lookup(bond, member_)->may_enable = may_enable;
+ member = bond_member_lookup(bond, member_);
+ if (member) {
+ member->may_enable = may_enable;
+ }
ovs_rwlock_unlock(&rwlock);
}
--
2.18.1
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
