[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20251203180924.6862bd26@gandalf.local.home>
Date: Wed, 3 Dec 2025 18:09:24 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: LKML <linux-kernel@...r.kernel.org>
Cc: John 'Warthog9' Hawley <warthog9@...nel.org>, "John W. Krahn"
<jwkrahn@...w.ca>
Subject: [PATCH] ktest.pl: Fix uninitialized var in config-bisect.pl
From: Steven Rostedt <rostedt@...dmis.org>
The error path of copying the old config used the wrong variable in the
error message:
$ mkdir /tmp/build
$ ./tools/testing/ktest/config-bisect.pl -b /tmp/build config-good /tmp/config-bad
$ chmod 0 /tmp/build
$ ./tools/testing/ktest/config-bisect.pl -b /tmp/build config-good /tmp/config-bad good
cp /tmp/build//.config config-good.tmp ... [0 seconds] FAILED!
Use of uninitialized value $config in concatenation (.) or string at ./tools/testing/ktest/config-bisect.pl line 744.
failed to copy to config-good.tmp
When it should have shown:
failed to copy /tmp/build//.config to config-good.tmp
Reported-by: "John W. Krahn" <jwkrahn@...w.ca>
Signed-off-by: Steven Rostedt <rostedt@...dmis.org>
---
[
Better late than never. John reported this to me back in 2022, but I kept
getting distracted before pushing it forward. :-p
I just came across it again in my INBOX (I never archived that email)
But my INBOX has 8000 messages in it, thus it probably has more of these :-/
]
tools/testing/ktest/config-bisect.pl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/testing/ktest/config-bisect.pl b/tools/testing/ktest/config-bisect.pl
index 6fd864935319..bee7cb34a289 100755
--- a/tools/testing/ktest/config-bisect.pl
+++ b/tools/testing/ktest/config-bisect.pl
@@ -741,9 +741,9 @@ if ($start) {
die "Can not find file $bad\n";
}
if ($val eq "good") {
- run_command "cp $output_config $good" or die "failed to copy $config to $good\n";
+ run_command "cp $output_config $good" or die "failed to copy $output_config to $good\n";
} elsif ($val eq "bad") {
- run_command "cp $output_config $bad" or die "failed to copy $config to $bad\n";
+ run_command "cp $output_config $bad" or die "failed to copy $output_config to $bad\n";
}
}
--
2.51.0
Powered by blists - more mailing lists