summaryrefslogtreecommitdiffstats
path: root/hw/omap_lcdc.c
diff options
context:
space:
mode:
authorMichael Tokarev2012-03-11 14:52:59 +0100
committerMichael Tokarev2012-06-07 18:43:26 +0200
commit45270ad8a86a80cca4c59dfa73d9a9ee0688d781 (patch)
tree9629b9598d8afc49cf215250d12ce8a217fb962d /hw/omap_lcdc.c
parenttarget-microblaze: fix swx build breakage (diff)
downloadqemu-45270ad8a86a80cca4c59dfa73d9a9ee0688d781.tar.gz
qemu-45270ad8a86a80cca4c59dfa73d9a9ee0688d781.tar.xz
qemu-45270ad8a86a80cca4c59dfa73d9a9ee0688d781.zip
virtio-serial-bus: use correct lengths in control_out() message
Original code has one thing to process (cur_len), requests to convert from iovec to buf another thing (len which is actually max_len), and processes something else (copied). Whole thing is very difficult to understand, even if it does a right thing. The iov_to_buf() conversion in this case will always return cur_len, because it is the length of the iovec it was asked to process, and the size we asked to convert is the same or larger, and iov_to_buf() will stop at reaching either iov or buf. Make the code saner by doing the only sane thing: dropping `copied' which is always the same as `cur_len' but just introduces questions. Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'hw/omap_lcdc.c')
0 files changed, 0 insertions, 0 deletions