Bypass ASAN leak checks in help and version command paths for FUSE 2

Restrict the _exit(EXIT_SUCCESS) workaround to FUSE versions older than
3.0.  This prevents LeakSanitizer from reporting a false positive
64-byte leak in libfuse2's fuse_chan_new while allowing modern FUSE 3
environments to use the standard C++ exit procedure. Verified that FUSE
3 does not exhibit the leak.

Author: fdegros

mount-zip.cc

@@ -490,20 +490,28 @@ static int ProcessArg(void* data,
         outargs->argv[0] = empty;
         fuse_main(outargs->argc, outargs->argv, &operations, nullptr);
         outargs->argv[0] = argv0;
+        std::exit(EXIT_SUCCESS);
 #else
+        dup2(STDOUT_FILENO, STDERR_FILENO);
         fuse_opt_add_arg(outargs, "-ho");  // I think ho means "help output".
         fuse_main(outargs->argc, outargs->argv, &operations, nullptr);
+        _exit(EXIT_SUCCESS);
 #endif
       }
-      std::exit(EXIT_SUCCESS);
 
     case KEY_VERSION:
       std::cout << PROGRAM_NAME " version: " PROGRAM_VERSION "\n"
                 << "libzip version: " LIBZIP_VERSION "\n"
                 << std::flush;
       fuse_opt_add_arg(outargs, "--version");
+#if FUSE_USE_VERSION >= 30
       fuse_main(outargs->argc, outargs->argv, &operations, nullptr);
       std::exit(EXIT_SUCCESS);
+#else
+      dup2(STDOUT_FILENO, STDERR_FILENO);
+      fuse_main(outargs->argc, outargs->argv, &operations, nullptr);
+      _exit(EXIT_SUCCESS);
+#endif
 
     case FUSE_OPT_KEY_NONOPT:
       if (param.paths.emplace_back(arg).empty()) {

tests/node_test.cc

@@ -13,11 +13,12 @@
 // You should have received a copy of the GNU General Public License
 // along with this program.  If not, see <https://www.gnu.org/licenses/>.
 
+#include "node.h"
+
 #include <gtest/gtest.h>
 #include <sys/stat.h>
 #include <iostream>
 #include <sstream>
-#include "node.h"
 
 namespace {
 
@@ -93,11 +94,11 @@ TEST(NodeTest, GetUniqueChildDirectory) {
       root.children.push_front(file);
 
       EXPECT_EQ(root.GetUniqueChildDirectory(), nullptr);
-      root.children.pop_front(); // remove file
+      root.children.pop_front();  // remove file
     }
 
     EXPECT_EQ(root.GetUniqueChildDirectory(), &child);
-    root.children.pop_front(); // remove child
+    root.children.pop_front();  // remove child
   }
 
   EXPECT_EQ(root.GetUniqueChildDirectory(), nullptr);
@@ -122,4 +123,4 @@ TEST(NodeTest, GetStat) {
   EXPECT_EQ(st.st_mode, static_cast<mode_t>(S_IFREG | 0755));
 }
 
-} // namespace
+}  // namespace