On Wed, Apr 28, 2010 at 2:09 PM, Serge E. Hallyn <[email protected]> wrote: > Signed-off-by: Serge Hallyn <[email protected]> > --- > testcases/kernel/containers/mqns/mqns_02.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/testcases/kernel/containers/mqns/mqns_02.c > b/testcases/kernel/containers/mqns/mqns_02.c > index 8f783ba..07ab042 100644 > --- a/testcases/kernel/containers/mqns/mqns_02.c > +++ b/testcases/kernel/containers/mqns/mqns_02.c > @@ -76,7 +76,7 @@ int check_mqueue(void *vtest) > > else { > > - if (read(p1[0], buf, 5)) > + if (read(p1[0], buf, 5) < 0) > tst_resm(TBROK | TERRNO, > "read(p1[0], ..) failed"); > else { > @@ -142,7 +142,7 @@ int main(int argc, char *argv[]) > } > > if (read(p2[0], buf, 7) < 0) { > - tst_resm(TBROK, "read(p1[0], ..) failed"); > + tst_resm(TBROK, "read(p2[0], ..) failed"); > } else if (!strcmp(buf, "mqfail")) { > tst_resm(TFAIL, "child process could not create mqueue\n"); > umount(DEV_MQUEUE); > @@ -160,8 +160,8 @@ int main(int argc, char *argv[]) > tst_resm(TFAIL, "Parent process found mqueue\n"); > mq_close(mqd); > } > - if (write(p1[1], "cont", 5)) { > - tst_resm(TBROK, "read(p1[0], ..) failed"); > + if (write(p1[1], "cont", 5) < 0) { > + tst_resm(TBROK, "write(p1[1], ..) failed");
Acked-by: Garrett Cooper <[email protected]> ------------------------------------------------------------------------------ _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
