<html><head></head><body><div style="color:#000; background-color:#fff; font-family:Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif;font-size:16px"><div id="yui_3_16_0_ym19_1_1491722327461_6476"><span id="yui_3_16_0_ym19_1_1491722327461_6504">Hi,</span></div><div id="yui_3_16_0_ym19_1_1491722327461_6450"><span><br></span></div><div id="yui_3_16_0_ym19_1_1491722327461_6452"><span id="yui_3_16_0_ym19_1_1491722327461_6451">Thanks Alexey, this would be much simpler. I'll send new patch. (also setpriority01 fails on my read-only system, I'll update it similiarly)<br><br></span></div><div id="yui_3_16_0_ym19_1_1491722327461_6453"><span id="yui_3_16_0_ym19_1_1491722327461_6471">Thanks a lot for your help,</span></div><div id="yui_3_16_0_ym19_1_1491722327461_6454"><span><br></span></div><div id="yui_3_16_0_ym19_1_1491722327461_6455"><span id="yui_3_16_0_ym19_1_1491722327461_6505">Best regards,</span></div><div id="yui_3_16_0_ym19_1_1491722327461_6458"><span><br></span></div><div id="yui_3_16_0_ym19_1_1491722327461_6457"><span>Hoang</span></div><div class="qtdSeparateBR" id="yui_3_16_0_ym19_1_1491722327461_6409"><br><br></div><div class="yahoo_quoted" id="yui_3_16_0_ym19_1_1491722327461_6359" style="display: block;"> <div style="font-family: Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif; font-size: 16px;" id="yui_3_16_0_ym19_1_1491722327461_6358"> <div style="font-family: HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, Sans-Serif; font-size: 16px;" id="yui_3_16_0_ym19_1_1491722327461_6357"> <div dir="ltr" id="yui_3_16_0_ym19_1_1491722327461_6356"> <font size="2" face="Arial" id="yui_3_16_0_ym19_1_1491722327461_6355"> <hr size="1"> <b><span style="font-weight:bold;">From:</span></b> Alexey Kodanev <alexey.kodanev@oracle.com><br> <b><span style="font-weight: bold;">To:</span></b> the_hoang0709@yahoo.com; ltp@lists.linux.it <br> <b id="yui_3_16_0_ym19_1_1491722327461_6507"><span style="font-weight: bold;" id="yui_3_16_0_ym19_1_1491722327461_6506">Sent:</span></b> Thursday, 6 April 2017, 0:25<br> <b id="yui_3_16_0_ym19_1_1491722327461_6509"><span style="font-weight: bold;" id="yui_3_16_0_ym19_1_1491722327461_6508">Subject:</span></b> Re: [LTP] [PATCH] prot_hsymlinks: Fix failure on read-only machine<br> </font> </div> <div class="y_msg_container" id="yui_3_16_0_ym19_1_1491722327461_6411"><br><div dir="ltr" id="yui_3_16_0_ym19_1_1491722327461_6410">Hi,<br clear="none">On 10.03.2017 3:54, <a shape="rect" ymailto="mailto:the_hoang0709@yahoo.com" href="mailto:the_hoang0709@yahoo.com">the_hoang0709@yahoo.com</a> wrote:<div class="yqt0838919780" id="yqtfd34887"><br clear="none">> ...<br clear="none">> if (disable_protected_hlinks) {<br clear="none">> tst_resm(TINFO, "Disable protected hardlinks mode back");<br clear="none">> @@ -316,8 +317,13 @@ static void test_user_cmd(const char *user_cmd)<br clear="none">> char cmd[MAX_CMD_LEN];<br clear="none">> snprintf(cmd, MAX_CMD_LEN, "%s %s", user_cmd, users[TEST_USER].name);<br clear="none">> if (system(cmd) != 0) {<br clear="none">> - tst_brkm(TBROK, cleanup, "Failed to run cmd: %s %s",<br clear="none">> + if (eaccess("/etc/passwd", W_OK)) {<br clear="none">> + tst_brkm(TCONF, cleanup,<br clear="none">> + "/etc/passwd is not accessible");</div><br clear="none"><br clear="none">It would be better to add this check to the top of setup(), right after<br clear="none">tst_kvercmp(), and exit without doing cleanup.<br clear="none"><br clear="none">Thanks,<br clear="none">Alexey<div class="yqt0838919780" id="yqtfd39567"><br clear="none"><br clear="none"></div></div><br><br></div> </div> </div> </div></div></body></html>