diff options
author | Ben Pfaff <blp@ovn.org> | 2016-01-06 13:38:20 -0800 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-01-19 22:50:33 -0800 |
commit | afe8a6c869d75d035b7550599234186b120a1afd (patch) | |
tree | 1883c28d431478de0a40c4d71fce9460683e67c0 /include | |
parent | 33fa5fe5819b00b7e57b95405ccf9a2e8a4cd258 (diff) | |
download | openvswitch-afe8a6c869d75d035b7550599234186b120a1afd.tar.gz |
ofp-msgs: Fix definitions of OF1.4 OFPT_GET_ASYNC_REPLY and OFPT_SET_ASYNC.
The structures declared in ofp-msgs.h for messages definitions should not
include an OpenFlow header (its presence is implied), but the definition of
these messages did. This commit fixes the definitions.
The visible bug was really minor here: messages of these kinds without any
TLVs would be rejected by the OpenFlow parser. But OVS never sends these
messages without TLVs, so probably no one ever noticed this. (Also, the
OVS support for OF1.4 is still incomplete and experimental.)
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Jarno Rajahalme <jarno@ovn.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/openflow/openflow-1.4.h | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/include/openflow/openflow-1.4.h b/include/openflow/openflow-1.4.h index e600cff68..daf6cf42e 100644 --- a/include/openflow/openflow-1.4.h +++ b/include/openflow/openflow-1.4.h @@ -245,17 +245,6 @@ struct ofp14_async_config_prop_header { ovs_be16 length; /* Length in bytes of this property. */ }; OFP_ASSERT(sizeof(struct ofp14_async_config_prop_header) == 4); - -/* Asynchronous message configuration. - * OFPT_GET_ASYNC_REPLY or OFPT_SET_ASYNC. - */ -struct ofp14_async_config { - struct ofp_header header; - /* Async config Property list - 0 or more */ - struct ofp14_async_config_prop_header properties[0]; -}; -OFP_ASSERT(sizeof(struct ofp14_async_config) == 8); - /* Request forward reason */ enum ofp14_requestforward_reason { OFPRFR_GROUP_MOD = 0, /* Forward group mod requests. */ |