[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/9] xen/efi: address violations of MISRA C:2012 Rule 11.8
On 14/12/23 17:32, Jan Beulich wrote:
On 14.12.2023 13:07, Simone Ballarin wrote:
--- a/xen/common/version.c
+++ b/xen/common/version.c
@@ -178,7 +178,7 @@ void __init xen_build_init(void)
if ( &n[1] >= __note_gnu_build_id_end )
return;
- sz = (void *)__note_gnu_build_id_end - (void *)n;
+ sz = (const void *)__note_gnu_build_id_end - (const void *)n;
rc = xen_build_id_check(n, sz, &build_id_p, &build_id_len);
How does this change fit the subject? I'm also inclined to ask that these
casts be changed to unsigned long or uintptr_t.
For the actual EFI change:
Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
Jan
Ok, v2 will use uintptr_t.
--
Simone Ballarin, M.Sc.
Field Application Engineer, BUGSENG (https://bugseng.com)
|
