* [LTP] [PATCH] Fix the bug of the temp-files removing
@ 2013-07-16 6:41 Jungsoo Son
2013-07-16 12:50 ` chrubis
0 siblings, 1 reply; 2+ messages in thread
From: Jungsoo Son @ 2013-07-16 6:41 UTC (permalink / raw)
To: ltp-list
The shell command to remove all the temp-files created had a bug that
try to remove /dev/null. If the ltp is run with root permission the
/dev/null will be removed.
Signed-off-by: jungsoo.son <jungsoo.son@lge.com>
---
testscripts/test_robind.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/testscripts/test_robind.sh b/testscripts/test_robind.sh
index 142678a..4c01f4e 100755
--- a/testscripts/test_robind.sh
+++ b/testscripts/test_robind.sh
@@ -193,7 +193,7 @@ function testdir
echo >> $PASSLOG
fi
# Remove all the temp-files created.
- eval rm -rf ${TMPDIR}/${dir}/* > /dev/null 2>&1 /dev/null || true
+ eval rm -rf ${TMPDIR}/${dir}/* > /dev/null 2>&1 || true
unset TDIRECTORY
popd
}
--
1.7.9.5
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [LTP] [PATCH] Fix the bug of the temp-files removing
2013-07-16 6:41 [LTP] [PATCH] Fix the bug of the temp-files removing Jungsoo Son
@ 2013-07-16 12:50 ` chrubis
0 siblings, 0 replies; 2+ messages in thread
From: chrubis @ 2013-07-16 12:50 UTC (permalink / raw)
To: Jungsoo Son; +Cc: ltp-list
Hi!
> The shell command to remove all the temp-files created had a bug that
> try to remove /dev/null. If the ltp is run with root permission the
> /dev/null will be removed.
Good catch, applied.
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-07-16 12:49 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-16 6:41 [LTP] [PATCH] Fix the bug of the temp-files removing Jungsoo Son
2013-07-16 12:50 ` chrubis
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox