diff options
| author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2016-10-27 19:20:01 +0000 |
|---|---|---|
| committer | Gurucharan Shetty <guru@ovn.org> | 2016-11-04 08:06:50 -0700 |
| commit | 583d34678d32482d616fe5bd7dc323f044788898 (patch) | |
| tree | 13c126e5df0f25864f72c62515b38fa8979d028d /lib/netdev-windows.c | |
| parent | 8ebddc5ffd5fbb89186e5f4aba5e3fd791ee7e4d (diff) | |
| download | openvswitch-583d34678d32482d616fe5bd7dc323f044788898.tar.gz | |
netdev-windows: fix ofpbuf initialization
ofpbuf_const_initializer takes only two parameters see:
https://github.com/openvswitch/ovs/blob/master/include/openvswitch/ofpbuf.h#L96
the compiler only gives the following warning:
lib/netdev-windows.c(229) : warning C4020: 'ofpbuf_const_initializer' :
too many actual parameters
which in result will put junk data into "ofpbuf b".
This patch fixes the initialization.
Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Acked-by: Sairam Venugopal <vsairam@vmware.com>
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Diffstat (limited to 'lib/netdev-windows.c')
| -rw-r--r-- | lib/netdev-windows.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/netdev-windows.c b/lib/netdev-windows.c index 32184906d..423a00b41 100644 --- a/lib/netdev-windows.c +++ b/lib/netdev-windows.c @@ -226,7 +226,7 @@ netdev_windows_netdev_from_ofpbuf(struct netdev_windows_netdev_info *info, netdev_windows_info_init(info); - struct ofpbuf b = ofpbuf_const_initializer(&b, buf->data, buf->size); + struct ofpbuf b = ofpbuf_const_initializer(buf->data, buf->size); struct nlmsghdr *nlmsg = ofpbuf_try_pull(&b, sizeof *nlmsg); struct genlmsghdr *genl = ofpbuf_try_pull(&b, sizeof *genl); struct ovs_header *ovs_header = ofpbuf_try_pull(&b, sizeof *ovs_header); |