From 767039c492ab9d1a38adc129273ea72b465e9c60 Mon Sep 17 00:00:00 2001
From: Antonio SJ Musumeci <trapexit@spawn.link>
Date: Thu, 10 Sep 2020 21:58:23 -0400
Subject: [PATCH] option_parser: return 0 when requesting help or version

---
 src/option_parser.cpp | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/option_parser.cpp b/src/option_parser.cpp
index 6878dcd4..7b9d33c6 100644
--- a/src/option_parser.cpp
+++ b/src/option_parser.cpp
@@ -129,7 +129,8 @@ namespace l
     else
       {
         l::read_config(data_,is);
-
+        if(is.fail())
+          data_->errs->push_back("failure reading config file");
         is.close();
       }
   }
@@ -426,17 +427,13 @@ option_processor(void       *data_,
 
     case MERGERFS_OPT_HELP:
       usage();
-      close(2);
-      dup(1);
-      fuse_opt_add_arg(outargs_,"-ho");
-      break;
+      exit(0);
 
     case MERGERFS_OPT_VERSION:
       std::cout << "mergerfs version: "
                 << (MERGERFS_VERSION[0] ? MERGERFS_VERSION : "unknown")
                 << std::endl;
-      fuse_opt_add_arg(outargs_,"--version");
-      break;
+      exit(0);
 
     default:
       break;
@@ -470,6 +467,11 @@ namespace options
                    opts,
                    ::option_processor);
 
+    if(config_->branches.vec.empty())
+      errs_->push_back("branches not set");
+    if(config_->mount->empty())
+      errs_->push_back("mountpoint not set");
+
     set_default_options(args_);
     set_fsname(args_,config_);
     set_subtype(args_);