Merge "Unmount public sdcard filesystems before killing the fuse process." into nyc-dev am: a4330be

am: a54cd87

* commit 'a54cd8739c8d313b66354f99ed9419ccc3f6be10':
  Unmount public sdcard filesystems before killing the fuse process.

Change-Id: I11813de6b10a9a98af823443332bb07b82cc1d06

PublicVolume.cpp

@@ -194,12 +194,10 @@ status_t PublicVolume::doMount() {
 }
 
 status_t PublicVolume::doUnmount() {
-    if (mFusePid > 0) {
-        kill(mFusePid, SIGTERM);
-        TEMP_FAILURE_RETRY(waitpid(mFusePid, nullptr, 0));
-        mFusePid = 0;
-    }
-
+    // Unmount the storage before we kill the FUSE process. If we kill
+    // the FUSE process first, most file system operations will return
+    // ENOTCONN until the unmount completes. This is an exotic and unusual
+    // error code and might cause broken behaviour in applications.
     KillProcessesUsingPath(getPath());
 
     ForceUnmount(kAsecPath);
@@ -209,6 +207,12 @@ status_t PublicVolume::doUnmount() {
     ForceUnmount(mFuseWrite);
     ForceUnmount(mRawPath);
 
+    if (mFusePid > 0) {
+        kill(mFusePid, SIGTERM);
+        TEMP_FAILURE_RETRY(waitpid(mFusePid, nullptr, 0));
+        mFusePid = 0;
+    }
+
     rmdir(mFuseDefault.c_str());
     rmdir(mFuseRead.c_str());
     rmdir(mFuseWrite.c_str());