diff options
| author | Michael S. Tsirkin <mst@redhat.com> | 2015-02-25 15:20:01 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-02-27 21:53:44 +0100 |
| commit | 0d79a493e507437a2135e5ac1a447d4d503488d8 (patch) | |
| tree | eeda1dacebc689cf8e39801af7218f1c5a10fb29 /drivers/vhost | |
| parent | vhost: cleanup iterator update logic (diff) | |
| download | linux-0d79a493e507437a2135e5ac1a447d4d503488d8.tar.xz linux-0d79a493e507437a2135e5ac1a447d4d503488d8.zip | |
vhost: drop hard-coded num_buffers size
The 2 that we use for copy_to_iter comes from sizeof(u16),
it used to be that way before the iov iter update.
Fix it up, making it obvious the size of stack access
is right.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/vhost')
| -rw-r--r-- | drivers/vhost/net.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/vhost/net.c b/drivers/vhost/net.c index ca7043459e2c..2bbfc25e582c 100644 --- a/drivers/vhost/net.c +++ b/drivers/vhost/net.c @@ -621,7 +621,8 @@ static void handle_rx(struct vhost_net *net) num_buffers = cpu_to_vhost16(vq, headcount); if (likely(mergeable) && - copy_to_iter(&num_buffers, 2, &fixup) != 2) { + copy_to_iter(&num_buffers, sizeof num_buffers, + &fixup) != sizeof num_buffers) { vq_err(vq, "Failed num_buffers write"); vhost_discard_vq_desc(vq, headcount); break; |