From 95e5339490f3aa323e80d91f524cc85459d0a88b Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Thu, 15 Oct 2020 18:28:42 +0000 Subject: [PATCH] nvme: fix signed/unsigned error with cuse ioctls cuse defines the ioctl handler to take an int for the command, but the kernel uses unsigned int. So cast the int parameter to unsigned int when doing the switch. Fixes issue #1613. Signed-off-by: Jim Harris Change-Id: I7ef13afc755ee79d8a79b3c7a09dccee7bb7aa1c Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4716 Tested-by: SPDK CI Jenkins Reviewed-by: Aleksey Marchuk Reviewed-by: Changpeng Liu Reviewed-by: Shuhei Matsumoto --- lib/nvme/nvme_cuse.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/nvme/nvme_cuse.c b/lib/nvme/nvme_cuse.c index 0dc4effb3..c0bee52b9 100644 --- a/lib/nvme/nvme_cuse.c +++ b/lib/nvme/nvme_cuse.c @@ -555,7 +555,7 @@ cuse_ctrlr_ioctl(fuse_req_t req, int cmd, void *arg, return; } - switch (cmd) { + switch ((unsigned int)cmd) { case NVME_IOCTL_ADMIN_CMD: cuse_nvme_admin_cmd(req, cmd, arg, fi, flags, in_buf, in_bufsz, out_bufsz); break; @@ -580,7 +580,7 @@ cuse_ns_ioctl(fuse_req_t req, int cmd, void *arg, return; } - switch (cmd) { + switch ((unsigned int)cmd) { case NVME_IOCTL_ADMIN_CMD: cuse_nvme_admin_cmd(req, cmd, arg, fi, flags, in_buf, in_bufsz, out_bufsz); break;