[kexec-tools] Resolves: bz 710744

Neil Horman nhorman at fedoraproject.org
Mon Jun 6 19:47:11 UTC 2011


commit fe18fead61fb07bdf1e70e94083c1132e4a18346
Author: Neil Horman <nhorman at tuxdriver.com>
Date:   Mon Jun 6 15:43:54 2011 -0400

    Resolves: bz 710744

 kexec-tools.spec |    5 ++++-
 mkdumprd         |    5 +++--
 2 files changed, 7 insertions(+), 3 deletions(-)
---
diff --git a/kexec-tools.spec b/kexec-tools.spec
index 864b02d..5cf6acb 100644
--- a/kexec-tools.spec
+++ b/kexec-tools.spec
@@ -1,6 +1,6 @@
 Name: kexec-tools
 Version: 2.0.0 
-Release: 46%{?dist}
+Release: 47%{?dist}
 License: GPLv2
 Group: Applications/System
 Summary: The kexec/kdump userspace component.
@@ -282,6 +282,9 @@ done
 
 
 %changelog
+* Mon Jun 02 2011 Neil Horman <nhorman at redhat.com> - 2.0.0-47
+- Fixed misuse of readlink command after directory change (bz 710744)
+
 * Tue Apr 26 2011 Neil Horman <nhorman at redhat.com> - 2.0.0-46
 - Fix some grammer in man page (bz 673817)
 
diff --git a/mkdumprd b/mkdumprd
index 26c5e2c..0abe921 100644
--- a/mkdumprd
+++ b/mkdumprd
@@ -364,7 +364,8 @@ findstoragedriverinsys () {
         [ "$PWD" = "/sys" ] && return
         cd ..
     done
-    cd $(readlink ./device)
+    devname=$(readlink ./device)
+    cd $devname
     while [ ! -f modalias ]; do
         [ "$PWD" = "/sys/devices" ] && return
         cd ..
@@ -372,7 +373,7 @@ findstoragedriverinsys () {
 
     [ -e modalias ] && modalias=$(cat modalias)
     [ -e device ] && modalias=$(cat $(find /sys/devices -name modalias
-        | grep $(basename $(readlink ./device))))
+        | grep $(basename $devname)))
 
     for driver in $(modprobe --set-version $kernel --show-depends $modalias 2>/dev/null| awk '{ print gensub(".*/","","g",$2) }') ; do
         findmodule ${driver%%.ko}


More information about the scm-commits mailing list