diff options
| author | Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com> | 2024-01-05 19:39:59 +0300 |
|---|---|---|
| committer | Kees Cook <keescook@chromium.org> | 2024-02-01 21:06:42 +0300 |
| commit | e03d4910e6e45cb49f630258e870b08f2ee34e7a (patch) | |
| tree | 09c2b0d2788cae483561eb89b52b4bc62e29a4d7 /drivers/misc/vmw_vmci | |
| parent | 09ce61e27db83180993e8b1a7f511af62374383c (diff) | |
| download | linux-e03d4910e6e45cb49f630258e870b08f2ee34e7a.tar.xz | |
VMCI: Use struct_size() in kmalloc()
Use struct_size() instead of open coding.
Suggested-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Signed-off-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/20240105164001.2129796-1-harshit.m.mogalapalli@oracle.com
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'drivers/misc/vmw_vmci')
| -rw-r--r-- | drivers/misc/vmw_vmci/vmci_datagram.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/misc/vmw_vmci/vmci_datagram.c b/drivers/misc/vmw_vmci/vmci_datagram.c index f50d22882476..ac6cb0c8d99b 100644 --- a/drivers/misc/vmw_vmci/vmci_datagram.c +++ b/drivers/misc/vmw_vmci/vmci_datagram.c @@ -224,8 +224,8 @@ static int dg_dispatch_as_host(u32 context_id, struct vmci_datagram *dg) return VMCI_ERROR_NO_MEM; } - dg_info = kmalloc(sizeof(*dg_info) + - (size_t) dg->payload_size, GFP_ATOMIC); + dg_info = kmalloc(struct_size(dg_info, msg_payload, dg->payload_size), + GFP_ATOMIC); if (!dg_info) { atomic_dec(&delayed_dg_host_queue_size); vmci_resource_put(resource); |