diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-01-07 07:30:08 +0100 |
|---|---|---|
| committer | Alex Williamson <alex.williamson@redhat.com> | 2017-01-11 20:12:37 +0100 |
| commit | 73da4268fdbae972f617946d1c690f2136964802 (patch) | |
| tree | 8a7c3341dd0456146c5e93d91a3dbd8b33f7b30a /samples | |
| parent | vfio-mdev: buffer overflow in ioctl() (diff) | |
| download | linux-73da4268fdbae972f617946d1c690f2136964802.tar.xz linux-73da4268fdbae972f617946d1c690f2136964802.zip | |
vfio-mdev: remove some dead code
We set info.count to 1 in mtty_get_irq_info() so static checkers
complain that, "Why do we have impossible conditions?" The answer is
that it seems to be left over dead code that can be safely removed.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Kirti Wankhede <kwankhede@nvidia.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'samples')
| -rw-r--r-- | samples/vfio-mdev/mtty.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/samples/vfio-mdev/mtty.c b/samples/vfio-mdev/mtty.c index 382f4797428f..ca495686b9c3 100644 --- a/samples/vfio-mdev/mtty.c +++ b/samples/vfio-mdev/mtty.c @@ -1230,9 +1230,6 @@ static long mtty_ioctl(struct mdev_device *mdev, unsigned int cmd, if (ret) return ret; - if (info.count == -1) - return -EINVAL; - if (copy_to_user((void __user *)arg, &info, minsz)) return -EFAULT; |