X-Git-Url: https://git.ozlabs.org/?p=ccan;a=blobdiff_plain;f=ccan%2Ffailtest%2Ffailtest.c;h=24c394aa993099024a804a8a1b71161ac9e015ee;hp=1dad0864aa372fa3b7ead07a081f9e125f192bbc;hb=eaff14657e211975f22d86210b98524401bcd45b;hpb=e43dc84fe91f64d8761aeda6338a1afbf701b7f5

diff --git a/ccan/failtest/failtest.c b/ccan/failtest/failtest.c
index 1dad0864..24c394aa 100644
--- a/ccan/failtest/failtest.c
+++ b/ccan/failtest/failtest.c
@@ -488,10 +488,39 @@ static NORETURN void failtest_cleanup(bool forced_cleanup, int status)
 	}
 
 	free_everything();
-	tell_parent(SUCCESS);
+	if (status == 0)
+		tell_parent(SUCCESS);
+	else
+		tell_parent(FAILURE);
 	exit(status);
 }
 
+static bool following_path(void)
+{
+	if (!failpath)
+		return false;
+	/* + means continue after end, like normal. */
+	if (*failpath == '+') {
+		failpath = NULL;
+		return false;
+	}
+	return true;
+}
+
+static bool follow_path(struct failtest_call *call)
+{
+	if (*failpath == '\0') {
+		/* Continue, but don't inject errors. */
+		return call->fail = false;
+	}
+
+	if (tolower((unsigned char)*failpath) != info_to_arg[call->type])
+		errx(1, "Failpath expected '%s' got '%c'\n",
+		     failpath, info_to_arg[call->type]);
+	call->fail = cisupper(*(failpath++));
+	return call->fail;
+}
+
 static bool should_fail(struct failtest_call *call)
 {
 	int status;
@@ -505,22 +534,8 @@ static bool should_fail(struct failtest_call *call)
 	if (call == &unrecorded_call)
 		return false;
 
-	if (failpath) {
-		/* + means continue after end, like normal. */
-		if (*failpath == '+')
-			failpath = NULL;
-		else if (*failpath == '\0') {
-			/* Continue, but don't inject errors. */
-			return call->fail = false;
-		} else {
-			if (tolower((unsigned char)*failpath)
-			    != info_to_arg[call->type])
-				errx(1, "Failpath expected '%c' got '%c'\n",
-				     info_to_arg[call->type], *failpath);
-			call->fail = cisupper(*(failpath++));
-			return call->fail;
-		}
-	}
+	if (following_path())
+		return follow_path(call);
 
 	/* Attach debugger if they asked for it. */
 	if (debugpath) {
@@ -847,6 +862,8 @@ int failtest_open(const char *pathname,
 	p->u.open.ret = open(pathname, call.flags, call.mode);
 
 	if (p->u.open.ret == -1) {
+		if (following_path())
+			follow_path(p);
 		p->fail = false;
 		p->error = errno;
 	} else if (should_fail(p)) {
@@ -1087,10 +1104,11 @@ int failtest_close(int fd, const char *file, unsigned line)
 	p = add_history(FAILTEST_CLOSE, file, line, &call);
 	p->fail = false;
 
-	/* Consume close from failpath. */
-	if (failpath)
-		if (should_fail(p))
+	/* Consume close from failpath (shouldn't tell us to fail). */
+	if (following_path()) {
+		if (follow_path(p))
 			abort();
+	}
 
 	if (fd < 0)
 		return close(fd);