Resolves: BZ1484945
https://bugzilla.redhat.com/show_bug.cgi?id=1484945
Currently the kdumpctl script doesn't handle
whitespaces (including TABs) which might be there before
an option name in the kdump.conf
This patch addresses this issue, by ensuring that the
kdumpctl errors out in case it finds any stray whitespaces
before a option name.
While at it, we also add a relevant note in the
kdump.conf man page to explain this error case.
Reported-by: Kenneth Dsouzak <kdsouza(a)redhat.com>
Signed-off-by: Bhupesh Sharma <bhsharma(a)redhat.com>
---
kdump.conf.5 | 4 ++++
kdumpctl | 9 ++++++++-
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/kdump.conf.5 b/kdump.conf.5
index 11b1fad559b4..a68e6e8f426a 100644
--- a/kdump.conf.5
+++ b/kdump.conf.5
@@ -339,6 +339,10 @@ scp /proc/vmcore <user@host>:path/vmcore
examples for other options please see
.I /etc/kdump.conf
+.PP
+.B
+NOTE: No whitespaces are allowed before a kdump OPTION name.
+
.SH SEE ALSO
kexec(8) mkdumprd(8) dracut.cmdline(7)
diff --git a/kdumpctl b/kdumpctl
index b7a3105026eb..b4340840453b 100755
--- a/kdumpctl
+++ b/kdumpctl
@@ -406,6 +406,13 @@ check_config()
return 1
}
+ # Check if we have any leading spaces before the
+ # variable name in the kdump conf file
+ if grep '^[[:space:]]' $KDUMP_CONFIG_FILE; then
+ echo "No whitespaces are allowed before a kdump OPTION name. Refer
to kdump.conf manpage for details"
+ return 1
+ fi
+
while read config_opt config_val; do
case "$config_opt" in
\#* | "")
@@ -432,7 +439,7 @@ check_config()
check_default_config || return 1
check_fence_kdump_config || return 1
-
+
return 0
}
--
2.7.4