diff options
author | Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> | 2012-08-20 13:35:23 +0100 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@gmail.com> | 2012-09-14 08:40:32 +0100 |
commit | 06b5f36d052b540a59b52150582d65674199b2ce (patch) | |
tree | c3ecb857f9a9730930ade0995ad3f13ecf05ac50 /migration-fd.c | |
parent | c67f5dc10573687497f0f5c3aec19b15c35c63d7 (diff) | |
download | qemu-06b5f36d052b540a59b52150582d65674199b2ce.zip qemu-06b5f36d052b540a59b52150582d65674199b2ce.tar.gz qemu-06b5f36d052b540a59b52150582d65674199b2ce.tar.bz2 |
net: do not report queued packets as sent
Net send functions have a return value where 0 means the packet has not
been sent and will be queued. A non-zero value means the packet was
sent or an error caused the packet to be dropped.
This patch fixes two instances where packets are queued but we return
their size. This causes callers to believe the packets were sent. When
the caller uses the async send interface this creates a real problem
because the callback will be invoked for a packet that the caller
believed to be already sent. This bug can cause double-frees in the
caller.
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Diffstat (limited to 'migration-fd.c')
0 files changed, 0 insertions, 0 deletions