diff options
author | 2017-05-17 19:24:15 +0100 | |
---|---|---|
committer | 2017-06-20 19:11:35 -0500 | |
commit | ecf3411a121e7a653e309ff50a820ffa87c537f8 (patch) | |
tree | c76212d915c9751d44386dcbb47c78bb182a35fb /fs/cifs/misc.c | |
parent | CIFS: Set ->should_dirty in cifs_user_readv() (diff) | |
download | linux-ecf3411a121e7a653e309ff50a820ffa87c537f8.tar.gz linux-ecf3411a121e7a653e309ff50a820ffa87c537f8.tar.bz2 linux-ecf3411a121e7a653e309ff50a820ffa87c537f8.zip |
CIFS: check if pages is null rather than bv for a failed allocation
pages is being allocated however a null check on bv is being used
to see if the allocation failed. Fix this by checking if pages is
null.
Detected by CoverityScan, CID#1432974 ("Logically dead code")
Fixes: ccf7f4088af2dd ("CIFS: Add asynchronous context to support kernel AIO")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Signed-off-by: Steve French <smfrench@gmail.com>
Diffstat (limited to 'fs/cifs/misc.c')
-rw-r--r-- | fs/cifs/misc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/misc.c b/fs/cifs/misc.c index b08531977daa..3b147dc6af63 100644 --- a/fs/cifs/misc.c +++ b/fs/cifs/misc.c @@ -810,7 +810,7 @@ setup_aio_ctx_iter(struct cifs_aio_ctx *ctx, struct iov_iter *iter, int rw) if (!pages) { pages = vmalloc(max_pages * sizeof(struct page *)); - if (!bv) { + if (!pages) { kvfree(bv); return -ENOMEM; } |