[Bug,tools/28724] eu-elfclassify --no-stdin option is not effect

Message ID bug-28724-10460-nekgGwmxLh@http.sourceware.org/bugzilla/
State Committed
Headers
Series [Bug,tools/28724] eu-elfclassify --no-stdin option is not effect |

Commit Message

dje at sourceware dot org April 24, 2022, 3:53 p.m. UTC
  https://sourceware.org/bugzilla/show_bug.cgi?id=28724

Mark Wielaard <mark at klomp dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
                 CC|                            |mark at klomp dot org

--- Comment #1 from Mark Wielaard <mark at klomp dot org> ---
Thanks, that was obviously a typo in the source. Nobody noticed before because
--no-stdin is the default. But it did indeed not work when given explicitly.

Fixed with:

commit 6398e94e9ac9a170da088768198d4a2b6e989e19 (HEAD -> master, origin/master,
origin/HEAD)
Author: Mark Wielaard <mark@klomp.org>
Date:   Sun Apr 24 17:48:39 2022 +0200

    elfclassify: Fix --no-stdin flag

    The no-stdin option was using the wrong flag, classify_flag_stdin,
    instead of classify_flag_no_stdin.

    https://sourceware.org/bugzilla/show_bug.cgi?id=28724

    Signed-off-by: Mark Wielaard <mark@klomp.org>
  

Patch

diff --git a/src/ChangeLog b/src/ChangeLog
index 6ef81862..fd87ce2f 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@ 
+2022-04-24  Mark Wielaard  <mark@klomp.org>
+
+       * elfclassify.c (main): Use classify_flag_no_stdin for no-std in
options.
+
 2022-04-19  Mark Wielaard  <mark@klomp.org>

        * readelf.c (get_dyn_ents): Define dyn_mem outside the while loop.
diff --git a/src/elfclassify.c b/src/elfclassify.c
index 2f70b29a..25fe9a65 100644
--- a/src/elfclassify.c
+++ b/src/elfclassify.c
@@ -946,7 +946,7 @@  separated by newlines"), 2 },
       { "stdin0", classify_flag_stdin0, NULL, 0,
         N_("Also read file names to process from standard input, \
 separated by ASCII NUL bytes"), 2 },
-      { "no-stdin", classify_flag_stdin, NULL, 0,
+      { "no-stdin", classify_flag_no_stdin, NULL, 0,
         N_("Do not read files from standard input (default)"), 2 },
       { "compressed", 'z', NULL, 0,
        N_("Try to open compressed files or embedded (kernel) ELF images"),