Power Ops: Attempt to execute file path only

Adding a check to verify that the path to execute
is a file only and not a directory.

Pull request: https://github.com/vmware/open-vm-tools/pull/689

diff --git a/open-vm-tools/AUTHORS b/open-vm-tools/AUTHORS
index 156f5cc2eb378afee29004b0e587e2cc789367e3..98c86fd6ce5fea8d359765bc761c315d0fbfbaa5 100644 (file)
--- a/open-vm-tools/AUTHORS
+++ b/open-vm-tools/AUTHORS
@@ -91,3 +91,6 @@ Dirk Mueller    Detect the proto files for containerd grpc client on SUSE like s
 
 Jan Engelhardt  Fix build problems with grpc (at least) 1.54
                 - https://github.com/vmware/open-vm-tools/pull/664
+
+Yun Zheng Hu    Power Ops: Attempt to execute file path only
+                - https://github.com/vmware/open-vm-tools/pull/689

diff --git a/open-vm-tools/scripts/common/statechange.sh b/open-vm-tools/scripts/common/statechange.sh
index 40732519aaf6f2555810a41cf2739fe3deff8892..b3f16f8ce9334d061064889eb9d0ce23aae8c4d3 100644 (file)
--- a/open-vm-tools/scripts/common/statechange.sh
+++ b/open-vm-tools/scripts/common/statechange.sh
@@ -88,7 +88,7 @@ RunScripts() {
 
    if [ -d "$scriptDir" ]; then
       for scriptFile in "$scriptDir"/*; do
-         if [ -x "$scriptFile" ]; then
+         if [ -f "$scriptFile" -a -x "$scriptFile" ]; then
             "$scriptFile" $powerOp
             exitCode=`expr $exitCode \| $?`
          fi