| Message ID | 20210506184304.321645-3-vgoyal@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | virtiofs, dax: Fix couple of smatch warnings | expand |
diff --git a/fs/fuse/dax.c b/fs/fuse/dax.c index f06fdad3f7b1..1608b6606ef0 100644 --- a/fs/fuse/dax.c +++ b/fs/fuse/dax.c @@ -253,7 +253,10 @@ static int dmap_removemapping_list(struct inode *inode, unsigned int num, struct fuse_removemapping_one *remove_one, *ptr; struct fuse_removemapping_in inarg; struct fuse_dax_mapping *dmap; - int ret, i = 0, nr_alloc; + int ret = 0, i = 0, nr_alloc; + + if (!num) + return ret; nr_alloc = min_t(unsigned int, num, FUSE_REMOVEMAPPING_MAX_ENTRY); remove_one = kmalloc_array(nr_alloc, sizeof(*remove_one), GFP_NOFS);
Dan reported a smatch warning about "ret" being uninitialized. Fix it. fs/fuse/dax.c:197 dmap_removemapping_list() error: uninitialized symbol 'ret'. Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Vivek Goyal <vgoyal@redhat.com> --- fs/fuse/dax.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)