[LTP] [PATCH 3/4] mqns2: fix some error checking and reporting

["Serge E. Hallyn" <serue@us.ibm.com>]

Signed-off-by: Serge Hallyn <serue@us.ibm.com>
---
 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");
 
 		}
 	}
-- 
1.7.0


------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list