Commit ec394af5 authored by Petr Machata's avatar Petr Machata Committed by David S. Miller

switchdev: SWITCHDEV_OBJ_PORT_{VLAN, MDB}(): Sanitize

The two macros SWITCHDEV_OBJ_PORT_VLAN() and SWITCHDEV_OBJ_PORT_MDB()
expand to a container_of() call, yielding an appropriate container of
their sole argument. However, due to a name collision, the first
argument, i.e. the contained object pointer, is not the only one to get
expanded. The third argument, which is a structure member name, and
should be kept literal, gets expanded as well. The only safe way to use
these two macros is therefore to name the local variable passed to them
"obj".

To fix this, rename the sole argument of the two macros from
"obj" (which collides with the member name) to "OBJ". Additionally,
instead of passing "OBJ" to container_of() verbatim, parenthesize it, so
that a comma in the passed-in expression doesn't pollute the
container_of() invocation.
Signed-off-by: default avatarPetr Machata <petrm@mellanox.com>
Acked-by: default avatarJiri Pirko <jiri@mellanox.com>
Reviewed-by: default avatarIdo Schimmel <idosch@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2eb487c1
...@@ -95,8 +95,8 @@ struct switchdev_obj_port_vlan { ...@@ -95,8 +95,8 @@ struct switchdev_obj_port_vlan {
u16 vid_end; u16 vid_end;
}; };
#define SWITCHDEV_OBJ_PORT_VLAN(obj) \ #define SWITCHDEV_OBJ_PORT_VLAN(OBJ) \
container_of(obj, struct switchdev_obj_port_vlan, obj) container_of((OBJ), struct switchdev_obj_port_vlan, obj)
/* SWITCHDEV_OBJ_ID_PORT_MDB */ /* SWITCHDEV_OBJ_ID_PORT_MDB */
struct switchdev_obj_port_mdb { struct switchdev_obj_port_mdb {
...@@ -105,8 +105,8 @@ struct switchdev_obj_port_mdb { ...@@ -105,8 +105,8 @@ struct switchdev_obj_port_mdb {
u16 vid; u16 vid;
}; };
#define SWITCHDEV_OBJ_PORT_MDB(obj) \ #define SWITCHDEV_OBJ_PORT_MDB(OBJ) \
container_of(obj, struct switchdev_obj_port_mdb, obj) container_of((OBJ), struct switchdev_obj_port_mdb, obj)
void switchdev_trans_item_enqueue(struct switchdev_trans *trans, void switchdev_trans_item_enqueue(struct switchdev_trans *trans,
void *data, void (*destructor)(void const *), void *data, void (*destructor)(void const *),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment