ofproto-dpif: Don't configure BFD on missing ofports.
authorEthan Jackson <ethan@nicira.com>
Tue, 21 May 2013 02:07:12 +0000 (19:07 -0700)
committerEthan Jackson <ethan@nicira.com>
Tue, 21 May 2013 17:05:14 +0000 (10:05 -0700)
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>
ofproto/ofproto.c

index 28df181..ca1dc89 100644 (file)
@@ -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