Skip to content

Commit 5217bd5

Browse files
committed
ksmbd: use kvzalloc instead of kvmalloc
Use kvzalloc instead of kvmalloc. Reported-by: kernel test robot <[email protected]> Signed-off-by: Namjae Jeon <[email protected]>
1 parent 35b2f16 commit 5217bd5

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

smb2pdu.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,7 @@ int smb2_allocate_rsp_buf(struct ksmbd_work *work)
532532
if (le32_to_cpu(hdr->NextCommand) > 0)
533533
sz = large_sz;
534534

535-
work->response_buf = kvmalloc(sz, GFP_KERNEL | __GFP_ZERO);
535+
work->response_buf = kvzalloc(sz, GFP_KERNEL);
536536
if (!work->response_buf)
537537
return -ENOMEM;
538538

@@ -6656,7 +6656,7 @@ static noinline int smb2_read_pipe(struct ksmbd_work *work)
66566656
}
66576657

66586658
work->aux_payload_buf =
6659-
kvmalloc(rpc_resp->payload_sz, GFP_KERNEL | __GFP_ZERO);
6659+
kvmalloc(rpc_resp->payload_sz, GFP_KERNEL);
66606660
if (!work->aux_payload_buf) {
66616661
err = -ENOMEM;
66626662
goto out;
@@ -6808,7 +6808,7 @@ int smb2_read(struct ksmbd_work *work)
68086808
ksmbd_debug(SMB, "filename %pD, offset %lld, len %zu\n",
68096809
fp->filp, offset, length);
68106810

6811-
work->aux_payload_buf = kvmalloc(length, GFP_KERNEL | __GFP_ZERO);
6811+
work->aux_payload_buf = kvzalloc(length, GFP_KERNEL);
68126812
if (!work->aux_payload_buf) {
68136813
err = -ENOMEM;
68146814
goto out;
@@ -6957,7 +6957,7 @@ static ssize_t smb2_write_rdma_channel(struct ksmbd_work *work,
69576957
int ret;
69586958
ssize_t nbytes;
69596959

6960-
data_buf = kvmalloc(length, GFP_KERNEL | __GFP_ZERO);
6960+
data_buf = kvzalloc(length, GFP_KERNEL);
69616961
if (!data_buf)
69626962
return -ENOMEM;
69636963

transport_ipc.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ static struct ksmbd_ipc_msg *ipc_msg_alloc(size_t sz)
231231
struct ksmbd_ipc_msg *msg;
232232
size_t msg_sz = sz + sizeof(struct ksmbd_ipc_msg);
233233

234-
msg = kvmalloc(msg_sz, GFP_KERNEL | __GFP_ZERO);
234+
msg = kvzalloc(msg_sz, GFP_KERNEL);
235235
if (msg)
236236
msg->sz = sz;
237237
return msg;
@@ -270,7 +270,7 @@ static int handle_response(int type, void *payload, size_t sz)
270270
entry->type + 1, type);
271271
}
272272

273-
entry->response = kvmalloc(sz, GFP_KERNEL | __GFP_ZERO);
273+
entry->response = kvzalloc(sz, GFP_KERNEL);
274274
if (!entry->response) {
275275
ret = -ENOMEM;
276276
break;

vfs.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -681,7 +681,7 @@ static int ksmbd_vfs_stream_write(struct ksmbd_file *fp, char *buf, loff_t *pos,
681681
}
682682

683683
if (v_len < size) {
684-
wbuf = kvmalloc(size, GFP_KERNEL | __GFP_ZERO);
684+
wbuf = kvzalloc(size, GFP_KERNEL);
685685
if (!wbuf) {
686686
err = -ENOMEM;
687687
goto out;
@@ -1670,7 +1670,7 @@ ssize_t ksmbd_vfs_listxattr(struct dentry *dentry, char **list)
16701670
if (size <= 0)
16711671
return size;
16721672

1673-
vlist = kvmalloc(size, GFP_KERNEL | __GFP_ZERO);
1673+
vlist = kvzalloc(size, GFP_KERNEL);
16741674
if (!vlist)
16751675
return -ENOMEM;
16761676

0 commit comments

Comments
 (0)