* [LTP] [PATCH] android: pty01: Fix pty01 test for Android.
@ 2017-08-28 18:26 Sandeep Patil
2017-08-28 18:32 ` enh
0 siblings, 1 reply; 3+ messages in thread
From: Sandeep Patil @ 2017-08-28 18:26 UTC (permalink / raw)
To: ltp
The test fails because 'grantpt()' is a no-op in bionic and doesn't set
the /dev/pts/X mode to '020600' as expected by the test. The change
skips that check if __ANDROID__ is defined so the rest of the test(s)
can proceed and detect other failures if any.
Signed-off-by: Sandeep Patil <sspatil@google.com>
---
testcases/kernel/pty/pty01.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/testcases/kernel/pty/pty01.c b/testcases/kernel/pty/pty01.c
index cbcc65c97..9fdbf0841 100644
--- a/testcases/kernel/pty/pty01.c
+++ b/testcases/kernel/pty/pty01.c
@@ -99,9 +99,12 @@ static int test1(void)
tst_brkm(TBROK, NULL, "uid mismatch");
}
+ /* grantpt() is a no-op in bionic / Android. */
+#ifndef __ANDROID__
if (st.st_mode != (S_IFCHR | S_IRUSR | S_IWUSR | S_IWGRP)) {
tst_brkm(TBROK, NULL, "mode mismatch (mode=%o)", st.st_mode);
}
+#endif
slavefd = open(slavename, O_RDWR);
if (slavefd >= 0) {
--
2.14.1.342.g6490525c54-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [LTP] [PATCH] android: pty01: Fix pty01 test for Android.
2017-08-28 18:26 [LTP] [PATCH] android: pty01: Fix pty01 test for Android Sandeep Patil
@ 2017-08-28 18:32 ` enh
2017-08-29 0:19 ` Sandeep Patil
0 siblings, 1 reply; 3+ messages in thread
From: enh @ 2017-08-28 18:32 UTC (permalink / raw)
To: ltp
(should be __BIONIC__ rather than __ANDROID__ since this will be true for
host bionic too.)
On Aug 28, 2017 11:26, "Sandeep Patil" <sspatil@google.com> wrote:
> The test fails because 'grantpt()' is a no-op in bionic and doesn't set
> the /dev/pts/X mode to '020600' as expected by the test. The change
> skips that check if __ANDROID__ is defined so the rest of the test(s)
> can proceed and detect other failures if any.
>
> Signed-off-by: Sandeep Patil <sspatil@google.com>
> ---
> testcases/kernel/pty/pty01.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/testcases/kernel/pty/pty01.c b/testcases/kernel/pty/pty01.c
> index cbcc65c97..9fdbf0841 100644
> --- a/testcases/kernel/pty/pty01.c
> +++ b/testcases/kernel/pty/pty01.c
> @@ -99,9 +99,12 @@ static int test1(void)
> tst_brkm(TBROK, NULL, "uid mismatch");
> }
>
> + /* grantpt() is a no-op in bionic / Android. */
> +#ifndef __ANDROID__
> if (st.st_mode != (S_IFCHR | S_IRUSR | S_IWUSR | S_IWGRP)) {
> tst_brkm(TBROK, NULL, "mode mismatch (mode=%o)",
> st.st_mode);
> }
> +#endif
>
> slavefd = open(slavename, O_RDWR);
> if (slavefd >= 0) {
> --
> 2.14.1.342.g6490525c54-goog
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20170828/51fdbae5/attachment.html>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [LTP] [PATCH] android: pty01: Fix pty01 test for Android.
2017-08-28 18:32 ` enh
@ 2017-08-29 0:19 ` Sandeep Patil
0 siblings, 0 replies; 3+ messages in thread
From: Sandeep Patil @ 2017-08-29 0:19 UTC (permalink / raw)
To: ltp
On Mon, Aug 28, 2017 at 11:32:30AM -0700, enh wrote:
> (should be __BIONIC__ rather than __ANDROID__ since this will be true for
> host bionic too.)
Thanks, fixing this now in v2.
- ssp
>
> On Aug 28, 2017 11:26, "Sandeep Patil" <sspatil@google.com> wrote:
>
> > The test fails because 'grantpt()' is a no-op in bionic and doesn't set
> > the /dev/pts/X mode to '020600' as expected by the test. The change
> > skips that check if __ANDROID__ is defined so the rest of the test(s)
> > can proceed and detect other failures if any.
> >
> > Signed-off-by: Sandeep Patil <sspatil@google.com>
> > ---
> > testcases/kernel/pty/pty01.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/testcases/kernel/pty/pty01.c b/testcases/kernel/pty/pty01.c
> > index cbcc65c97..9fdbf0841 100644
> > --- a/testcases/kernel/pty/pty01.c
> > +++ b/testcases/kernel/pty/pty01.c
> > @@ -99,9 +99,12 @@ static int test1(void)
> > tst_brkm(TBROK, NULL, "uid mismatch");
> > }
> >
> > + /* grantpt() is a no-op in bionic / Android. */
> > +#ifndef __ANDROID__
> > if (st.st_mode != (S_IFCHR | S_IRUSR | S_IWUSR | S_IWGRP)) {
> > tst_brkm(TBROK, NULL, "mode mismatch (mode=%o)",
> > st.st_mode);
> > }
> > +#endif
> >
> > slavefd = open(slavename, O_RDWR);
> > if (slavefd >= 0) {
> > --
> > 2.14.1.342.g6490525c54-goog
> >
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-08-29 0:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-28 18:26 [LTP] [PATCH] android: pty01: Fix pty01 test for Android Sandeep Patil
2017-08-28 18:32 ` enh
2017-08-29 0:19 ` Sandeep Patil
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.