commit 1b32f85e54731e36ea7141c654afb5c3df2c493e
author yusukes <yusukes@google.com> Mon Mar 05 10:24:58 2018 -0800
committer Treehugger Robot <treehugger-gerrit@google.com> Thu Mar 22 02:40:37 2018 +0000
tree d15c74d8a298ec59779d077c5dd55aa008af9677
parent 059e0bd235a9a48e7c1ed8bd26a9af81ae23cbf0

Use warn() when logging an error for setup_mount_destination()

setup_mount_destination() does not always set |errno| before
returning. Also, setup_mount_destination() now has pwarn()s
for each failure path.

Bug: None
Test: unit tests still pass
Change-Id: Ie603cffb1b8ea28a5f92a4d790b92e281ff2c6a0

libminijail.c

diff --git a/libminijail.c b/libminijail.c
index f9fb0e9..c6e0d63 100644
--- a/libminijail.c
+++ b/libminijail.c
@@ -1375,7 +1375,7 @@
 	ret = setup_mount_destination(m->src, dest, j->uid, j->gid,
 				      (m->flags & MS_BIND));
 	if (ret) {
-		pwarn("creating mount target '%s' failed", dest);
+		warn("creating mount target '%s' failed", dest);
 		goto error;
 	}