|add STOP variable for debugging

Author: ryanofsky

src/ipc/interfaces.cpp

@@ -12,8 +12,10 @@
 #include <tinyformat.h>
 #include <util/system.h>
 
+#include <boost/algorithm/string.hpp>
 #include <functional>
 #include <memory>
+#include <signal.h>
 #include <stdexcept>
 #include <stdio.h>
 #include <stdlib.h>
@@ -25,6 +27,28 @@
 
 namespace ipc {
 namespace {
+//! Send stop signal to current process to aid debugging if directed by STOP
+//! environment variable.
+void DebugStop(const char* exe_name, int argc, char* argv[])
+{
+    FILE* gdb = fsbridge::fopen("/tmp/gdb.txt", "a");
+    fprintf(gdb, "%i %s\n", getpid(), argv[0]);
+    fclose(gdb);
+    if (const char* env_stop = getenv("STOP")) {
+        std::string stop = env_stop;
+        std::vector<std::string> stops;
+        if (stop.size()) boost::split(stops, stop, boost::is_space(), boost::token_compress_on);
+        for (const auto& s : stops) {
+            if (strstr(exe_name, s.c_str())) {
+                printf("Pid %i stopping for GDB\n", getpid());
+                printf("sudo gdb -ex c %s %i\n", argv[0], getpid());
+                raise(SIGSTOP);
+                break;
+            }
+        }
+    }
+}
+
 class IpcImpl : public interfaces::Ipc
 {
 public:
@@ -47,6 +71,7 @@ class IpcImpl : public interfaces::Ipc
     }
     bool startSpawnedProcess(int argc, char* argv[], int& exit_status) override
     {
+        DebugStop(m_exe_name, argc, argv);
         exit_status = EXIT_FAILURE;
         int32_t fd = -1;
         if (!m_process->checkSpawned(argc, argv, fd)) {

test/lint/lint-locale-dependence.sh

@@ -44,6 +44,8 @@ KNOWN_VIOLATIONS=(
     "src/dbwrapper.cpp:.*vsnprintf"
     "src/httprpc.cpp.*trim"
     "src/init.cpp:.*atoi"
+    "src/ipc/interfaces.cpp:.*is_space"
+    "src/ipc/interfaces.cpp:.*printf"
     "src/qt/rpcconsole.cpp:.*atoi"
     "src/rest.cpp:.*strtol"
     "src/test/dbwrapper_tests.cpp:.*snprintf"