This error code was just overlooked before.

Reported-by: Anup Khadka <khadka...@gmail.com>
Signed-off-by: Ben Pfaff <b...@nicira.com>
---
 lib/ofp-errors.h  | 2 +-
 ofproto/ofproto.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/ofp-errors.h b/lib/ofp-errors.h
index 342877d..aff2075 100644
--- a/lib/ofp-errors.h
+++ b/lib/ofp-errors.h
@@ -160,7 +160,7 @@ enum ofperr {
      *   code defined the specification. ] */
     OFPERR_OFPBRC_BAD_TABLE_ID,
 
-    /* OF1.2+(1,10).  Denied because controller is slave. */
+    /* OF1.0-1.1(1,5), OF1.2+(1,10).  Denied because controller is slave. */
     OFPERR_OFPBRC_IS_SLAVE,
 
     /* NX1.0-1.1(1,514), OF1.2+(1,11).  Invalid port.  [ A non-standard error
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index ba4263e..b3909ad 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -3110,7 +3110,7 @@ reject_slave_controller(struct ofconn *ofconn)
 {
     if (ofconn_get_type(ofconn) == OFCONN_PRIMARY
         && ofconn_get_role(ofconn) == OFPCR12_ROLE_SLAVE) {
-        return OFPERR_OFPBRC_EPERM;
+        return OFPERR_OFPBRC_IS_SLAVE;
     } else {
         return 0;
     }
-- 
2.1.3

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to