[PATCH] kdumpctl: fix a variable expansion in check_fence_kdump_config()

[Pingfan Liu]

Both $ipaddrs and $node can hold multiple strings, so use "" to brace them.

Signed-off-by: Pingfan Liu 
---
 kdumpctl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kdumpctl b/kdumpctl
index 3f6a9be..2e7f842 100755
--- a/kdumpctl
+++ b/kdumpctl
@@ -944,7 +944,7 @@ check_fence_kdump_config()
    		return 1
    	fi
    	# node can be ipaddr
-		echo $ipaddrs | grep $node > /dev/null
+		echo "$ipaddrs " | grep "$node " > /dev/null
    	if [ $? -eq 0 ]; then
    		derror "Option fence_kdump_nodes cannot contain $node"
    		return 1

[Kairui Song]

On Fri, Dec 11, 2020 at 9:22 AM Pingfan Liu  wrote:
> Both $ipaddrs and $node can hold multiple strings, so use "" to brace them.
> Signed-off-by: Pingfan Liu
>
> kdumpctl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kdumpctl b/kdumpctl
> index 3f6a9be..2e7f842 100755
> --- a/kdumpctl
> +++ b/kdumpctl
> @@ -944,7 +944,7 @@ check_fence_kdump_config()
>                         return 1
>                 fi
>                 # node can be ipaddr
>           echo $ipaddrs | grep $node > /dev/null
>           echo "$ipaddrs " | grep "$node " > /dev/null
>           if [ $? -eq 0 ]; then
>                   derror "Option fence_kdump_nodes cannot contain $node"
>                   return 1
>
> --
> 2.7.5
> _______________________________________________
> kexec mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]
> Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: https://lists.fedoraproject.org/archives/list/[e-mail]

Acked-by: Kairui Song