ofproto: Don't always treat passive controllers as "equal".
If a passive controller chooses to configure itself as a slave controller,
I don't know a reason why it should be considered "equal" for some
purposes.
Acked-by: Justin Pettit
@@ -3425,8 +3425,7 @@ handle_set_config(struct ofconn *ofconn, const struct ofp_header *oh)
static enum ofperr
reject_slave_controller(struct ofconn *ofconn)
{
- if (ofconn_get_type(ofconn) == OFCONN_PRIMARY
- && ofconn_get_role(ofconn) == OFPCR12_ROLE_SLAVE) {
+ if (ofconn_get_role(ofconn) == OFPCR12_ROLE_SLAVE) {
return OFPERR_OFPBRC_IS_SLAVE;
} else {
return 0;
