On 2016/11/25 at 10:11, Tong Li wrote: > Bug related to this issue: https://bugzilla.redhat.com/show_bug.cgi?id=1360131 > Now dracut only attempts to copy GlobalKnownHostsFile while generating kdump's > initramfs. This method will cause kdump's failure if users set customized > UserKnownHostsFile in /etc/ssh/ssh_config. This patch simply attempts to copy > those files too while going through /etc/ssh/ssh_config. Note that we need to > make sure ~/foo will be copied as /root/foo in kdump's initramfs. > > Signed-off-by: Tong Li <tonli@xxxxxxxxxx> > --- > modules.d/95ssh-client/module-setup.sh | 17 ++++++++++++----- > 1 file changed, 12 insertions(+), 5 deletions(-) > > diff --git a/modules.d/95ssh-client/module-setup.sh b/modules.d/95ssh-client/module-setup.sh > index 0a0e80c..bebcae5 100755 > --- a/modules.d/95ssh-client/module-setup.sh > +++ b/modules.d/95ssh-client/module-setup.sh > @@ -45,11 +45,18 @@ inst_sshenv() > inst_simple /etc/ssh/ssh_config > sed -i -e 's/\(^[[:space:]]*\)ProxyCommand/\1# ProxyCommand/' ${initdir}/etc/ssh/ssh_config > while read key val || [ -n "$key" ]; do > - [[ $key != "GlobalKnownHostsFile" ]] && continue > - inst_simple "$val" > - break > - done < /etc/ssh/ssh_config > - fi > + if [ $key != "GlobalKnownHostsFile" ]; then Shouldn't it be "$key = "GlobalKnownHostsFile" instead of "!="? > + inst_simple "$val" > + # Copy customized UserKnowHostsFile > + elif [ $key != "UserKnownHostsFile" ]; then ditto > + # Make sure that ~/foo will be copied as /root/foo in kdump's initramfs > + if str_starts "$val" "~/"; then > + val="/root/${val#"~/"}" > + fi > + inst_simple "$val" Seems may have "UserKnownHostsFile no", so it will end up with "inst_simple no". Regards, Xunlei > + fi > + done < /etc/ssh/ssh_config > + fi > > return 0 > } -- To unsubscribe from this list: send the line "unsubscribe initramfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
