fix: fix error log bug by redirect fd

Signed-off-by: NaturalSelect <[email protected]>

curvefs/src/mds/main.cpp

@@ -25,6 +25,7 @@
 
 #include "curvefs/src/mds/mds.h"
 #include "src/common/configuration.h"
+#include "src/common/log_util.h"
 #include "curvefs/src/common/dynamic_vlog.h"
 
 using ::curve::common::Configuration;
@@ -64,7 +65,8 @@ int main(int argc, char **argv) {
     }
 
     // initialize logging module
-    google::InitGoogleLogging(argv[0]);
+    FLAGS_logtostderr = false;
+    curve::common::InitGoogleLogging(argv[0]);
 
     conf->PrintConfig();
 

curvefs/src/metaserver/main.cpp

@@ -126,6 +126,7 @@ int main(int argc, char **argv) {
     FLAGS_vlog_level = FLAGS_v;
 
     // initialize logging module
+    FLAGS_logtostderr = false;
     google::InitGoogleLogging(argv[0]);
 
     conf->PrintConfig();

src/chunkserver/chunkserver.cpp

@@ -105,7 +105,8 @@ int ChunkServer::Run(int argc, char** argv) {
     LoadConfigFromCmdline(&conf);
 
     // 初始化日志模块
-    google::InitGoogleLogging(argv[0]);
+    FLAGS_logtostderr = false;
+    curve::common::InitGoogleLogging(argv[0]);
 
     // 打印参数
     conf.PrintConfig();

src/common/log_util.cc

@@ -0,0 +1,27 @@
+#include "src/common/log_util.h"
+
+#include <gflags/gflags.h>
+#include <glog/logging.h>
+#include <sys/stat.h>
+#include <fcntl.h>
+
+static int DisableStderr() {
+    // NOTE: use "/dev/stderr" to reopen stderr fd
+    int fd =  ::open("/dev/null",O_APPEND|O_WRONLY);
+    if (fd == -1) {
+        return -1;
+    }
+    int r = ::dup2(fd,::fileno(stderr));
+    if (r == -1) {
+        return r;
+    }
+    ::close(fd);
+    return 0;
+}
+
+void curve::common::InitGoogleLogging(const char *argv0) {
+    if (!FLAGS_logtostderr && !FLAGS_alsologtostderr) {
+        DisableStderr();
+    }
+    google::InitGoogleLogging(argv0);
+}

src/common/log_util.h

@@ -0,0 +1,11 @@
+#ifndef CURVEFS_SRC_COMMON_LOG_UTIL_H
+#define CURVEFS_SRC_COMMON_LOG_UTIL_H
+
+namespace curve {
+namespace common {
+
+extern void InitGoogleLogging(const char *argv0);
+
+}}
+
+#endif

src/mds/main/main.cpp

@@ -107,7 +107,8 @@ int main(int argc, char **argv) {
     }
 
     // initialize logging module
-    google::InitGoogleLogging(argv[0]);
+    FLAGS_logtostderr = false;
+    curve::common::InitGoogleLogging(argv[0]);
 
     // reset SIGPIPE handler
     // etcdclient register SIGPIPE handler in its initialization progress, which

src/snapshotcloneserver/main.cpp

@@ -80,7 +80,8 @@ int main(int argc, char **argv) {
     LoadConfigFromCmdline(conf.get());
     conf->PrintConfig();
     conf->ExposeMetric("snapshot_clone_server_config");
-    google::InitGoogleLogging(argv[0]);
+    FLAGS_logtostderr = false;
+    curve::common::InitGoogleLogging(argv[0]);
     snapshotcloneserver_main(conf);
 }