From: Rusty Russell <rusty@rustcorp.com.au>
Date: Tue, 25 Oct 2016 04:46:07 +0000 (+1030)
Subject: ptr_valid: fix warn-unused-result warning (-O2)
X-Git-Url: http://git.ozlabs.org/?p=ccan;a=commitdiff_plain;h=ca8bfcc49ad8b340d0de2081bd7955a16a07a6f2;hp=0ce52c7ca42c2b3a6458cac348f5a8b96d7f1b62

ptr_valid: fix warn-unused-result warning (-O2)

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
---

diff --git a/ccan/ptr_valid/ptr_valid.c b/ccan/ptr_valid/ptr_valid.c
index ef95d34f..dc61dd27 100644
--- a/ccan/ptr_valid/ptr_valid.c
+++ b/ccan/ptr_valid/ptr_valid.c
@@ -246,9 +246,14 @@ static bool check_with_child(struct ptr_valid_batch *batch,
 			return false;
 	}
 
-	write(batch->to_child, &p, sizeof(p));
-	write(batch->to_child, &size, sizeof(size));
-	write(batch->to_child, &is_write, sizeof(is_write));
+	if (write(batch->to_child, &p, sizeof(p))
+	    + write(batch->to_child, &size, sizeof(size))
+	    + write(batch->to_child, &is_write, sizeof(is_write))
+	    != sizeof(p) + sizeof(size) + sizeof(is_write)) {
+		finish_child(batch);
+		errno = EFAULT;
+		return false;
+	}
 
 	if (read(batch->from_child, &ret, sizeof(ret)) != sizeof(ret)) {
 		finish_child(batch);