Numerous register fields in the relocator state are simply not
used depending on the relocator. This causes Coverity to flag
these fields but there is no real bug here. Simply initializing
the variable to {0} solves the issue. Fixed in the else case too
for consistency.
Fixes: CID 396932
Signed-off-by: Ross Philipson <ross.philip...@oracle.com>
---
grub-core/loader/i386/bsd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/grub-core/loader/i386/bsd.c b/grub-core/loader/i386/bsd.c
index 5838fc8f4..1f9128f6f 100644
--- a/grub-core/loader/i386/bsd.c
+++ b/grub-core/loader/i386/bsd.c
@@ -728,7 +728,7 @@ grub_freebsd_boot (void)
if (is_64bit)
{
- struct grub_relocator64_state state;
+ struct grub_relocator64_state state = {0};
grub_uint8_t *pagetable;
grub_uint32_t *stack;
grub_addr_t stack_target;
@@ -767,7 +767,7 @@ grub_freebsd_boot (void)
}
else
{
- struct grub_relocator32_state state;
+ struct grub_relocator32_state state = {0};
grub_uint32_t *stack;
grub_addr_t stack_target;
--
2.37.1
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel
