From 9bd84d4044d23e7a828cb8493c664073ed5b1f69 Mon Sep 17 00:00:00 2001
From: Antoine Musso <hashar@free.fr>
Date: Wed, 24 Jun 2015 15:21:06 +0200
Subject: [PATCH] Only chown tmp dirs when they are a tmpfs mount

Temp dirs are created with mktemp and thus belong to the user. There
is no need to chown them unless we used `mount -t tmpfs`.

Move chown under the tmpfs_check conditional.

Change-Id: I37efe18ced3a06d461364dc5cb20600f1527e995
---
 lib/common-functions | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/common-functions b/lib/common-functions
index c0e7ffb9..3680da77 100644
--- a/lib/common-functions
+++ b/lib/common-functions
@@ -37,8 +37,8 @@ function mk_build_dir () {
   if tmpfs_check ; then
     sudo mount -t tmpfs tmpfs $TMP_BUILD_DIR
     sudo mount -t tmpfs tmpfs $TMP_IMAGE_DIR
+    sudo chown $(id -u):$(id -g) $TMP_BUILD_DIR $TMP_IMAGE_DIR
   fi
-  sudo chown $(id -u):$(id -g) $TMP_BUILD_DIR $TMP_IMAGE_DIR
   trap trap_cleanup EXIT
   echo Building in $TMP_BUILD_DIR
   export TMP_IMAGE_PATH=$TMP_IMAGE_DIR/image.raw