aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Vlasenko <vda.linux@googlemail.com>2007-11-25 04:54:13 +0000
committerDenis Vlasenko <vda.linux@googlemail.com>2007-11-25 04:54:13 +0000
commit6446c2d4aba02eef0d8553c882f8d235288a66d9 (patch)
tree1637549e2c7e6432e6ec36774d0e5aa48e0bf44e
parent72e76044cfda377486a5199a0d35d71edf669a42 (diff)
downloadbusybox-6446c2d4aba02eef0d8553c882f8d235288a66d9.tar.gz
which: fix testsuite failure
pidof: do not output empty line if nothing found testsuite: disable false positive
-rw-r--r--debianutils/which.c4
-rw-r--r--libbb/find_pid_by_name.c2
-rw-r--r--procps/pidof.c10
-rwxr-xr-xtestsuite/pidof.tests4
-rwxr-xr-xtestsuite/runtest1
5 files changed, 13 insertions, 8 deletions
diff --git a/debianutils/which.c b/debianutils/which.c
index 883813059..5ab67194d 100644
--- a/debianutils/which.c
+++ b/debianutils/which.c
@@ -22,11 +22,11 @@ int which_main(int argc, char **argv)
bb_show_usage();
}
-/* We shouldn't do this. Ever. Not our business.
+ /* This matches what is seen on e.g. ubuntu
+ * "which" there is a shell script */
if (!getenv("PATH")) {
putenv((char*)bb_PATH_root_path);
}
-*/
while (--argc > 0) {
argv++;
diff --git a/libbb/find_pid_by_name.c b/libbb/find_pid_by_name.c
index 1fafec83a..8dcdb13bc 100644
--- a/libbb/find_pid_by_name.c
+++ b/libbb/find_pid_by_name.c
@@ -63,7 +63,7 @@ pid_t* find_pid_by_name(const char* procName)
(!p->comm[sizeof(p->comm)-2] && strcmp(p->comm, procName) == 0)
/* or we require argv0 to match (essential for matching reexeced /proc/self/exe)*/
|| (p->argv0 && strcmp(bb_basename(p->argv0), procName) == 0)
- /* TOOD: we can also try exe, do we want that? */
+ /* TOOD: we can also try /proc/NUM/exe link, do we want that? */
) {
pidList = xrealloc(pidList, sizeof(*pidList) * (i+2));
pidList[i++] = p->pid;
diff --git a/procps/pidof.c b/procps/pidof.c
index f29561789..86d1957a6 100644
--- a/procps/pidof.c
+++ b/procps/pidof.c
@@ -47,12 +47,13 @@ int pidof_main(int argc, char **argv)
}
#endif
/* Looks like everything is set to go. */
- while (optind < argc) {
+ argv += optind;
+ while (*argv) {
pid_t *pidList;
pid_t *pl;
/* reverse the pidlist like GNU pidof does. */
- pidList = pidlist_reverse(find_pid_by_name(argv[optind]));
+ pidList = pidlist_reverse(find_pid_by_name(*argv));
for (pl = pidList; *pl; pl++) {
#if ENABLE_FEATURE_PIDOF_OMIT
if (opt & OPT_OMIT) {
@@ -74,9 +75,10 @@ int pidof_main(int argc, char **argv)
#endif
}
free(pidList);
- optind++;
+ argv++;
}
- bb_putchar('\n');
+ if (!first)
+ bb_putchar('\n');
#if ENABLE_FEATURE_PIDOF_OMIT
if (ENABLE_FEATURE_CLEAN_UP)
diff --git a/testsuite/pidof.tests b/testsuite/pidof.tests
index bfde26e2c..29cfa943a 100755
--- a/testsuite/pidof.tests
+++ b/testsuite/pidof.tests
@@ -22,7 +22,9 @@ optional FEATURE_PIDOF_SINGLE
testing "pidof -s" "pidof -s init" "1\n" "" ""
optional FEATURE_PIDOF_OMIT
-testing "pidof -o %PPID" "pidof -o %PPID pidof.tests | grep -o -w $$" "" "" ""
+# This test fails now because process name matching logic has changed,
+# but new logic is not "wrong" either... see find_pid_by_name.c comments
+#testing "pidof -o %PPID" "pidof -o %PPID pidof.tests | grep -o -w $$" "" "" ""
testing "pidof -o %PPID NOP" "pidof -o %PPID -s init" "1\n" "" ""
testing "pidof -o init" "pidof -o 1 init | grep -o -w 1" "" "" ""
diff --git a/testsuite/runtest b/testsuite/runtest
index 4946cbd0d..0f081f9a5 100755
--- a/testsuite/runtest
+++ b/testsuite/runtest
@@ -31,6 +31,7 @@ run_applet_testcase()
mkdir -p tmp
pushd tmp > /dev/null
+# echo Running testcase $testcase
d=$tsdir sh -x -e $testcase >.logfile.txt 2>&1 || status=$?
if [ $status -ne 0 ]; then