From 1927ccfe0acefd4f58fa43cd1318290be73889fb Mon Sep 17 00:00:00 2001
From: Robb Kistler <code@robbkistler.com>
Date: Tue, 15 Dec 2015 15:07:00 -0800
Subject: [PATCH 1/2] Don't loop on fusermount if mountpoint is gone

---
 test/small-integration-test.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/small-integration-test.sh b/test/small-integration-test.sh
index c6116e3..5742725 100755
--- a/test/small-integration-test.sh
+++ b/test/small-integration-test.sh
@@ -49,7 +49,7 @@ function exit_handler {
     then
         kill $S3PROXY_PID
     fi
-    retry 30 fusermount -u $TEST_BUCKET_MOUNT_POINT_1
+    retry 30 grep $TEST_BUCKET_MOUNT_POINT_1 /proc/mounts && fusermount -u $TEST_BUCKET_MOUNT_POINT_1
 }
 trap exit_handler EXIT
 

From b581290c30931488db3d7f6268de9959bb255c26 Mon Sep 17 00:00:00 2001
From: Robb Kistler <code@robbkistler.com>
Date: Tue, 15 Dec 2015 15:20:40 -0800
Subject: [PATCH 2/2] Cleanly exit fuse loop on error in s3fs_init

This allows FUSE to clean the mount point up, preventing
"Transport endpoint not connected" errors on subsequent
access to the mount.
---
 src/s3fs.cpp | 23 ++++++++++++++++++++---
 1 file changed, 20 insertions(+), 3 deletions(-)

diff --git a/src/s3fs.cpp b/src/s3fs.cpp
index 86490eb..a952555 100644
--- a/src/s3fs.cpp
+++ b/src/s3fs.cpp
@@ -89,6 +89,7 @@ bool foreground                   = false;
 bool nomultipart                  = false;
 bool pathrequeststyle             = false;
 bool is_specified_endpoint        = false;
+int  s3fs_init_deferred_exit_status = 0;
 std::string program_name;
 std::string service_path          = "/";
 std::string host                  = "http://s3.amazonaws.com";
@@ -3291,6 +3292,19 @@ static int s3fs_removexattr(const char* path, const char* name)
 
   return 0;
 }
+   
+// s3fs_init calls this function to exit cleanly from the fuse event loop.
+//
+// There's no way to pass an exit status to the high-level event loop API, so 
+// this function stores the exit value in a global for main()
+static void s3fs_exit_fuseloop(int exit_status) {
+    S3FS_PRN_ERR("Exiting FUSE event loop due to errors\n");
+    s3fs_init_deferred_exit_status = exit_status;
+    struct fuse_context *ctx = fuse_get_context();
+    if (NULL != ctx) {
+        fuse_exit(ctx->fuse);
+    }
+}
 
 static void* s3fs_init(struct fuse_conn_info* conn)
 {
@@ -3299,13 +3313,15 @@ static void* s3fs_init(struct fuse_conn_info* conn)
   // ssl init
   if(!s3fs_init_global_ssl()){
     S3FS_PRN_CRIT("could not initialize for ssl libraries.");
-    exit(EXIT_FAILURE);
+    s3fs_exit_fuseloop(EXIT_FAILURE);
+    return NULL;
   }
 
   // init curl
   if(!S3fsCurl::InitS3fsCurl("/etc/mime.types")){
     S3FS_PRN_CRIT("Could not initiate curl library.");
-    exit(EXIT_FAILURE);
+    s3fs_exit_fuseloop(EXIT_FAILURE);
+    return NULL;
   }
 
   if (create_bucket){
@@ -3318,7 +3334,8 @@ static void* s3fs_init(struct fuse_conn_info* conn)
   if(!S3fsCurl::IsPublicBucket()){
     int result;
     if(EXIT_SUCCESS != (result = s3fs_check_service())){
-      exit(result);
+      s3fs_exit_fuseloop(result);
+      return NULL;
     }
   }