This function is not used by netvsc driver yet.
Still the code should handle case where device driver returns
zero (due to rescind).
Coverity issue: 302871
Fixes:
831dba47bd36 ("bus/vmbus: add Hyper-V virtual bus support")
Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
int vmbus_uio_irq_read(struct rte_vmbus_device *dev)
{
int32_t count;
int vmbus_uio_irq_read(struct rte_vmbus_device *dev)
{
int32_t count;
-
- if (read(dev->intr_handle.fd, &count, sizeof(count)) < 0) {
- VMBUS_LOG(ERR, "cannot read to %d:%s",
- dev->intr_handle.fd, strerror(errno));
- count = -errno;
+ int cc;
+
+ cc = read(dev->intr_handle.fd, &count, sizeof(count));
+ if (cc < (int)sizeof(count)) {
+ if (cc < 0) {
+ VMBUS_LOG(ERR, "IRQ read failed %s",
+ strerror(errno));
+ return -errno;
+ }
+ VMBUS_LOG(ERR, "can't read IRQ count");
+ return -EINVAL;