fix a problem introduced by commit cfa955b083c5617212a29a03423e063ff6cb350a. (connmgr: Fix packet-in reason for OpenFlow1.3 table-miss flow entries.)
ofconn might not be connected here. in that case ofputil_protocol_to_ofp_version aborts. Signed-off-by: YAMAMOTO Takashi <[email protected]> --- ofproto/connmgr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ofproto/connmgr.c b/ofproto/connmgr.c index 3c2b6cc..c9feae5 100644 --- a/ofproto/connmgr.c +++ b/ofproto/connmgr.c @@ -1496,9 +1496,9 @@ wire_reason(struct ofconn *ofconn, const struct ofproto_packet_in *pin) { if (pin->generated_by_table_miss && pin->up.reason == OFPR_ACTION) { enum ofputil_protocol protocol = ofconn_get_protocol(ofconn); - enum ofp_version version = ofputil_protocol_to_ofp_version(protocol); - if (version >= OFP13_VERSION) { + if (protocol != OFPUTIL_P_NONE + && ofputil_protocol_to_ofp_version(protocol) >= OFP13_VERSION) { return OFPR_NO_MATCH; } } -- 1.8.3.1 _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
