[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 1/3] gdb: Fix redirection issue in dump_module_sections

From: Glenn Washburn
Subject: [PATCH 1/3] gdb: Fix redirection issue in dump_module_sections
Date: Tue, 21 Feb 2023 12:28:46 -0600

An error in any GDB command causes it to immediately abort with an error,
this includes any command that calls that command. This leads to an issue
in dump_module_sections where an error causes the command to exit without
turning off file redirection. The user then ends up with a GDB command
line where commands output nothing to the console.

Instead do the work of dump_module_sections in the command
dump_module_sections_helper and run the command using GDB's pipe command
which does the redirection and undoes the redirection when it finishes
regardless of any errors in the command.

Also, remove .segments.tmp file prior to loading modules in case one was
left from a previous run.

Signed-off-by: Glenn Washburn <>
 grub-core/ | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/grub-core/ b/grub-core/
index e322d3dc10..4e45ad5622 100644
--- a/grub-core/
+++ b/grub-core/
@@ -10,15 +10,8 @@
 # Add section numbers and addresses to .segments.tmp
-define dump_module_sections
+define dump_module_sections_helper
        set $mod = $arg0
-       # FIXME: save logging status
-       set logging file .segments.tmp
-       set logging redirect on
-       set logging overwrite off
-       set logging on
        printf "%s", $mod->name
        set $segment = $mod->segment
        while ($segment)
@@ -26,9 +19,10 @@ define dump_module_sections
                set $segment = $segment->next
        printf "\n"
-       set logging off
-       # FIXME: restore logging status
+define dump_module_sections
+       pipe dump_module_sections_helper $arg0 | sh -c 'cat >>.segments.tmp'
 document dump_module_sections
        Gather information about module whose mod structure was
@@ -59,6 +53,7 @@ document load_module
 define load_all_modules
+       shell rm -f .segments.tmp
        set $this = grub_dl_head
        while ($this != 0)
                dump_module_sections $this

reply via email to

[Prev in Thread] Current Thread [Next in Thread]