diff options
| author | Jason Wang <jasowang@redhat.com> | 2013-04-26 18:27:40 +0800 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2013-05-07 06:43:09 -0500 |
| commit | d7108d90100d5bac5965abef5ed73f2602adae14 (patch) | |
| tree | cbe64c9178d700172b24bc5c65b872161b094b72 /qga/main.c | |
| parent | fd8192a5a868ddbb711abbf4a871b09bbb1dae04 (diff) | |
| download | qemu-d7108d90100d5bac5965abef5ed73f2602adae14.tar.gz qemu-d7108d90100d5bac5965abef5ed73f2602adae14.tar.bz2 qemu-d7108d90100d5bac5965abef5ed73f2602adae14.zip | |
virtio-net: properly check the vhost status during status set
Commit 32993698 (vhost: disable on tap link down) tries to disable the vhost
also when the peer's link is down. But the check was not done properly, the
vhost were only started when:
1) peer's link is not down
2) virtio-net has already been started.
Since == have a higher precedence than &&, place a brace to make sure both the
conditions were met then does the check. This fixes the crash when doing a savem
after set the link off which let qemu crash and complains:
virtio_net_save: Assertion `!n->vhost_started' failed.
Cc: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
Message-id: 1366972060-21606-1-git-send-email-jasowang@redhat.com
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'qga/main.c')
0 files changed, 0 insertions, 0 deletions