Call grub_partition_get_name unconditionally to initialize the part
variable. Then part will only be NULL when grub_partition_get_name errors.
Note that when source->partition is NULL, then grub_partition_get_name
returns an allocated empty string. So no comma or partition will be printed,
as desired.

Signed-off-by: Glenn Washburn <developm...@efficientek.com>
---
 grub-core/disk/cryptodisk.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/grub-core/disk/cryptodisk.c b/grub-core/disk/cryptodisk.c
index 99265097a..2381c3330 100644
--- a/grub-core/disk/cryptodisk.c
+++ b/grub-core/disk/cryptodisk.c
@@ -1021,11 +1021,10 @@ grub_cryptodisk_scan_device_real (const char *name,
       {
        /* Get the passphrase from the user, if no key data. */
        askpass = 1;
-       if (source->partition != NULL)
-         part = grub_partition_get_name (source->partition);
+       part = grub_partition_get_name (source->partition);
        grub_printf_ (N_("Enter passphrase for %s%s%s (%s): "), source->name,
                     source->partition != NULL ? "," : "",
-                    part != NULL ? part : "",
+                    part != NULL ? part : N_("UNKNOWN"),
                     dev->uuid);
        grub_free (part);
 
-- 
2.27.0


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to