<div dir="ltr">(I have no idea why this didn't go out as a -v2, apologies, please disregard)<div><br></div><div>- ssp<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 28, 2017 at 5:21 PM, Sandeep Patil <span dir="ltr"><<a href="mailto:sspatil@google.com" target="_blank">sspatil@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">The test fails because 'grantpt()' is a no-op in bionic and doesn't set<br>
the /dev/pts/X mode to '020600' as expected by the test. The change<br>
</span>skips that check if __BIONIC__ is defined so the rest of the test(s)<br>
<span class="">can proceed and detect other failures if any.<br>
<br>
Signed-off-by: Sandeep Patil <<a href="mailto:sspatil@google.com">sspatil@google.com</a>><br>
---<br>
 testcases/kernel/pty/pty01.c | 3 +++<br>
 1 file changed, 3 insertions(+)<br>
<br>
diff --git a/testcases/kernel/pty/pty01.c b/testcases/kernel/pty/pty01.c<br>
</span>index cbcc65c97..283612d55 100644<br>
<span class="">--- a/testcases/kernel/pty/pty01.c<br>
+++ b/testcases/kernel/pty/pty01.c<br>
@@ -99,9 +99,12 @@ static int test1(void)<br>
                tst_brkm(TBROK, NULL, "uid mismatch");<br>
        }<br>
<br>
</span>+        /* grantpt() is a no-op in bionic. */<br>
+#ifndef __BIONIC__<br>
<div class="HOEnZb"><div class="h5">        if (st.st_mode != (S_IFCHR | S_IRUSR | S_IWUSR | S_IWGRP)) {<br>
                tst_brkm(TBROK, NULL, "mode mismatch (mode=%o)", st.st_mode);<br>
        }<br>
+#endif<br>
<br>
        slavefd = open(slavename, O_RDWR);<br>
        if (slavefd >= 0) {<br>
--<br>
2.14.1.342.g6490525c54-goog<br>
<br>
</div></div></blockquote></div><br></div></div></div>