datapath: Fix type of actions_len_left in modify_vlan_tci().
authorBen Pfaff <blp@nicira.com>
Mon, 27 Dec 2010 23:21:29 +0000 (15:21 -0800)
committerBen Pfaff <blp@nicira.com>
Tue, 28 Dec 2010 17:16:37 +0000 (09:16 -0800)
commit4faaba2b90979e1106b8c9b0430481f2cc62364d
treef3464dfc76b9e4e6510571f48b7a658879ff199f
parent7f7439e31523643f48479325a250cc92f66b9fc8
datapath: Fix type of actions_len_left in modify_vlan_tci().

The sparse checker reported that the type of the pointer passed to
nla_next(), as &actions_len_left, was incorrect: whereas the parameter
has type "int *", &actions_len_left is an "unsigned int *".  This fixes
the problem.  It is not a bug fix since the code is equally correct (or
incorrect) either way, but it gets the types right anyhow.

I don't know why GCC was not reporting this as an error.

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/actions.c