datapath: Fix behavior of NLA_NESTED for pre-2.6.29 kernels.
authorBen Pfaff <blp@nicira.com>
Mon, 31 Jan 2011 23:47:43 +0000 (15:47 -0800)
committerBen Pfaff <blp@nicira.com>
Tue, 1 Feb 2011 05:40:20 +0000 (21:40 -0800)
commit0700107651b6a774f8f7ba873259a5d5011e3cb0
treebd365288aab22510517425a663d8797a5cfa9c39
parentd2a23af25132de097060d59f94465c94138c78a1
datapath: Fix behavior of NLA_NESTED for pre-2.6.29 kernels.

Before v2.6.29, a NLA_NESTED attribute, if it was present, was not allowed
to be empty.  However, OVS depends on the ability to accept empty
attributes.  For example, a present but empty ODP_FLOW_ATTR_ACTIONS on
ODP_FLOW_CMD_SET replaces the existing set of actions by an empty "drop"
action, whereas a missing ODP_FLOW_ATTR_ACTIONS leaves the existing
actions, if any, unchanged.

NLA_NESTED is different from NLA_UNSPEC in only two ways:

* If the size of the nested attributes is zero, no further size checks
  are performed.

* If the size of the nested attributes is not zero and no length
  parameter is specified the minimum size of nested attributes is
  NLA_HDRLEN.

nla_parse_nested() validates that there is at least enough space for
NLA_HDRLEN, so neither of these conditions are important, and we might
as well use NLA_UNSPEC with old kernels.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/linux-2.6/compat-2.6/include/net/netlink.h