github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-22 20:54:03 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

Xen/gnttab: adjust two uses of sizeof()

Browse Source 
The use in gnttab_map() is latently buggy, as "frames" there is
xen_pfn_t *, not unsigned long *. Adjust the correct use in
gnttab_map_frames_v2() as well, just to avoid the problematic pattern of
sizeof(<type>).

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
Signed-off-by: Juergen Gross <jgross@suse.com>

Message-ID: <0ddff6c4-7ec7-41a9-9417-687ca16aaafd@suse.com>
This commit is contained in:
Jan Beulich 2026-07-30 16:57:58 +02:00 committed by Juergen Gross
parent 80c0cc4766
commit 5d519a058c
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/xen/grant-table.c
View File

@ -1406,7 +1406,7 @@ static int gnttab_map_frames_v2(xen_pfn_t *frames, unsigned int nr_gframes)
/* No need for kzalloc as it is initialized in following hypercall
* GNTTABOP_get_status_frames.
*/
sframes = kmalloc_array(nr_sframes, sizeof(uint64_t), GFP_ATOMIC);
sframes = kmalloc_array(nr_sframes, sizeof(*sframes), GFP_ATOMIC);
if (!sframes)
return -ENOMEM;
@ -1478,7 +1478,7 @@ static int gnttab_map(unsigned int start_idx, unsigned int end_idx)
/* No need for kzalloc as it is initialized in following hypercall
* GNTTABOP_setup_table.
*/
frames = kmalloc_array(nr_gframes, sizeof(unsigned long), GFP_ATOMIC);
frames = kmalloc_array(nr_gframes, sizeof(*frames), GFP_ATOMIC);
if (!frames)
return -ENOMEM;