vhost: check file descriptor before closing

This avoids closing -1 in our case.

Signed-off-by: Huawei Xie <huawei.xie@intel.com>
Acked-by: Changchun Ouyang <changchun.ouyang@intel.com>
Acked-by: Tetsuya Mukawa <mukawa@igel.co.jp>
This commit is contained in:
Huawei Xie 2015-03-06 19:05:43 +08:00 committed by Thomas Monjalon
parent 64ab971791
commit 4d16fff496

View File

@ -185,13 +185,13 @@ cleanup_device(struct virtio_net *dev)
}
/* Close any event notifiers opened by device. */
if (dev->virtqueue[VIRTIO_RXQ]->callfd)
if ((int)dev->virtqueue[VIRTIO_RXQ]->callfd >= 0)
close((int)dev->virtqueue[VIRTIO_RXQ]->callfd);
if (dev->virtqueue[VIRTIO_RXQ]->kickfd)
if ((int)dev->virtqueue[VIRTIO_RXQ]->kickfd >= 0)
close((int)dev->virtqueue[VIRTIO_RXQ]->kickfd);
if (dev->virtqueue[VIRTIO_TXQ]->callfd)
if ((int)dev->virtqueue[VIRTIO_TXQ]->callfd >= 0)
close((int)dev->virtqueue[VIRTIO_TXQ]->callfd);
if (dev->virtqueue[VIRTIO_TXQ]->kickfd)
if ((int)dev->virtqueue[VIRTIO_TXQ]->kickfd >= 0)
close((int)dev->virtqueue[VIRTIO_TXQ]->kickfd);
}