Hi,<div id="yMail_cursorElementTracker_1490151719455"><br></div><div id="yMail_cursorElementTracker_1490151720554">Could you help to check, so that we can close the Issue ?</div><div id="yMail_cursorElementTracker_1490151799358"><br></div><div id="yMail_cursorElementTracker_1490151800372">Thanks and regards</div><div id="yMail_cursorElementTracker_1490151805541"><br></div><div id="yMail_cursorElementTracker_1490151806675">Hoang<br><br><div id="ymail_android_signature"><a href="https://overview.mail.yahoo.com/mobile/?.src=Android">Sent from Yahoo Mail on Android</a></div> <br> <blockquote style="margin: 0 0 20px 0;"> <header style="font-family:Roboto, sans-serif; color:#6D00F6;"> <div>On Fri, Mar 10, 2017 at 7:55, the_hoang0709@yahoo.com</div><div><the_hoang0709@yahoo.com> wrote:</div> </header> <div style="padding: 10px 0 0 20px; margin: 10px 0 0 0; border-left: 1px solid #6D00F6;"> <div dir="ltr">From: Hoang Nguyen <<a ymailto="mailto:the_hoang0709@yahoo.com" href="mailto:the_hoang0709@yahoo.com">the_hoang0709@yahoo.com</a>><br></div><div dir="ltr"><br></div><div dir="ltr">Return 32 if /etc/passwd is read-only.<br></div><div dir="ltr">Also move tst_tmpdir before useradd in setup, to avoid "TESTDIR was<br></div><div dir="ltr">NULL" in tst_rmdir in cleanup.<br></div><div dir="ltr"><br></div><div dir="ltr">Fixes #88<br></div><div dir="ltr"><br></div><div dir="ltr">Signed-off-by: Hoang Nguyen <<a ymailto="mailto:the_hoang0709@yahoo.com" href="mailto:the_hoang0709@yahoo.com">the_hoang0709@yahoo.com</a>><br></div><div dir="ltr">---<br></div><div dir="ltr"> testcases/kernel/security/prot_hsymlinks/prot_hsymlinks.c | 14 ++++++++++----<br></div><div dir="ltr"> 1 file changed, 10 insertions(+), 4 deletions(-)<br></div><div dir="ltr"><br></div><div dir="ltr">diff --git a/testcases/kernel/security/prot_hsymlinks/prot_hsymlinks.c b/testcases/kernel/security/prot_hsymlinks/prot_hsymlinks.c<br></div><div dir="ltr">index b23064a..31ddefa 100644<br></div><div dir="ltr">--- a/testcases/kernel/security/prot_hsymlinks/prot_hsymlinks.c<br></div><div dir="ltr">+++ b/testcases/kernel/security/prot_hsymlinks/prot_hsymlinks.c<br></div><div dir="ltr">@@ -206,6 +206,8 @@ static void setup(int argc, char *argv[])<br></div><div dir="ltr"> <br></div><div dir="ltr">     tst_sig(FORK, DEF_HANDLER, cleanup);<br></div><div dir="ltr"> <br></div><div dir="ltr">+    tst_tmpdir();<br></div><div dir="ltr">+<br></div><div dir="ltr">     test_user_cmd("useradd");<br></div><div dir="ltr">     /*<br></div><div dir="ltr">      * enable hardlinks and symlinks restrictions,<br></div><div dir="ltr">@@ -220,8 +222,6 @@ static void setup(int argc, char *argv[])<br></div><div dir="ltr">         disable_protected_slinks = 1;<br></div><div dir="ltr">     }<br></div><div dir="ltr"> <br></div><div dir="ltr">-    tst_tmpdir();<br></div><div dir="ltr">-<br></div><div dir="ltr">     init_base_dirs();<br></div><div dir="ltr"> <br></div><div dir="ltr">     init_files_dirs();<br></div><div dir="ltr">@@ -273,7 +273,8 @@ static void cleanup(void)<br></div><div dir="ltr">     if (skip_cleanup)<br></div><div dir="ltr">         return;<br></div><div dir="ltr"> <br></div><div dir="ltr">-    test_user_cmd("userdel -r");<br></div><div dir="ltr">+    if (!eaccess("/etc/passwd", W_OK))<br></div><div dir="ltr">+        test_user_cmd("userdel -r");<br></div><div dir="ltr"> <br></div><div dir="ltr">     if (disable_protected_hlinks) {<br></div><div dir="ltr">         tst_resm(TINFO, "Disable protected hardlinks mode back");<br></div><div dir="ltr">@@ -316,8 +317,13 @@ static void test_user_cmd(const char *user_cmd)<br></div><div dir="ltr">     char cmd[MAX_CMD_LEN];<br></div><div dir="ltr">     snprintf(cmd, MAX_CMD_LEN, "%s %s", user_cmd, users[TEST_USER].name);<br></div><div dir="ltr">     if (system(cmd) != 0) {<br></div><div dir="ltr">-        tst_brkm(TBROK, cleanup, "Failed to run cmd: %s %s",<br></div><div dir="ltr">+        if (eaccess("/etc/passwd", W_OK)) {<br></div><div dir="ltr">+            tst_brkm(TCONF, cleanup,<br></div><div dir="ltr">+            "/etc/passwd is not accessible");<br></div><div dir="ltr">+        } else {<br></div><div dir="ltr">+            tst_brkm(TBROK, cleanup, "Failed to run cmd: %s %s",<br></div><div dir="ltr">             user_cmd, users[TEST_USER].name);<br></div><div dir="ltr">+        }<br></div><div dir="ltr">     }<br></div><div dir="ltr"> }<br></div><div dir="ltr"> <br></div><div dir="ltr">-- <br></div><div dir="ltr">2.7.4<br></div><div dir="ltr"><br></div> </div> </blockquote></div>