From: Ethan Jackson <ethan@nicira.com>
Date: Tue, 21 May 2013 02:07:12 +0000 (-0700)
Subject: ofproto-dpif: Don't configure BFD on missing ofports.
X-Git-Tag: sliver-openvswitch-1.10.90-3~6^2~235
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=e8999bdc671c60775ba410bcd5d12e383276d703;p=sliver-openvswitch.git

ofproto-dpif: Don't configure BFD on missing ofports.

This was causing ovs-vswitchd to core occasionally, even when BFD
wasn't turned on.

Bug #17162.
Signed-off-by: Ethan Jackson <ethan@nicira.com>
---

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 28df181b9..ca1dc897a 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -812,6 +812,7 @@ ofproto_port_set_bfd(struct ofproto *ofproto, uint16_t ofp_port,
     if (!ofport) {
         VLOG_WARN("%s: cannot configure bfd on nonexistent port %"PRIu16,
                   ofproto->name, ofp_port);
+        return;
     }
 
     error = (ofproto->ofproto_class->set_bfd