]> code.ossystems Code Review - openembedded-core.git/commit
base-files: remove ending colon from default $PATH for root user
authorKevin Tian <kevin.tian@intel.com>
Wed, 26 Jan 2011 08:28:45 +0000 (16:28 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 28 Jan 2011 16:49:12 +0000 (16:49 +0000)
commitabc25cb88570bf54d3bb518b5083402894cca061
treeea24890e4acbbe6f7163645d536d5fd1c861de2f
parent1b26bce9675f1beb3d0370f49fc5ed9709c979f6
base-files: remove ending colon from default $PATH for root user

now the PATH for root user defined in a problematic way
    PATH=$PATH:/usr/local/sbin:/usr/sbin:/sbin:

from eglibc:
    /* Two adjacent colons, or a colon at the beginning or the end
       of `PATH' means to search the current directory.  */

that means current directory is always searched as the last path,
which could generate undesired behavior.

One example is found in LTP cron_deny01 test, which always complains
    "sh: cron_deny01 not found"

cron_deny01 is a shell script which setups the initial test preparation
and then invokes itself for real test under a different user:
    su $TEST_USER1 -c "$0"

'su' doesn't inherit PATH into the sub-shell, and thus $0 has to
be an absolute path to have right script found.

ltp appends the path of cron_deny01 to $PATH before running the test:
    export PATH="${PATH}:${LTPROOT}/testcases/bin"

In ideal way "${LTPROOT}/testcases/bin/cron_deny01" is found and becomes
$0, which works well.

However due to the ending colon in original PATH:
    PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin:/sbin::${LTPROOT}/testcases/bin
$0 becomes 'cron_deny01' w/o leading path which makes sub-shell under 'su'
failed to locate cron_deny01.

remove ending colon then fixes this problem.

Signed-off-by: Kevin Tian <kevin.tian@intel.com>
meta/recipes-core/base-files/base-files/profile
meta/recipes-core/base-files/base-files_3.0.14.bb