Large refactoring of dinitctl.
[oweals/dinit.git] / src / dinitctl.cc
index 4bead220a1f9c46977e17f4f43bd34585bf07c42..bd56180f3fb3e590a692da62380ce1788ae9237b 100644 (file)
@@ -4,11 +4,13 @@
 #include <string>
 #include <iostream>
 #include <system_error>
+#include <memory>
 
 #include <sys/types.h>
 #include <sys/socket.h>
 #include <sys/un.h>
 #include <unistd.h>
+#include <signal.h>
 #include <pwd.h>
 
 #include "control-cmds.h"
@@ -17,7 +19,7 @@
 
 // dinitctl:  utility to control the Dinit daemon, including starting and stopping of services.
 
-// This utility communicates with the dinit daemon via a unix socket (/dev/initctl).
+// This utility communicates with the dinit daemon via a unix stream socket (/dev/initctl, or $HOME/.dinitctl).
 
 using handle_t = uint32_t;
 
@@ -29,15 +31,34 @@ class ReadCPException
     ReadCPException(int err) : errcode(err) { }
 };
 
-static void fillBufferTo(CPBuffer *buf, int fd, int rlength)
+static int issueLoadService(int socknum, const char *service_name);
+static int checkLoadReply(int socknum, CPBuffer<1024> &rbuffer, handle_t *handle_p, ServiceState *state_p);
+static int startStopService(int socknum, const char *service_name, int command, bool do_pin, bool wait_for_service, bool verbose);
+static int unpinService(int socknum, const char *service_name, bool verbose);
+
+
+// Fill a circular buffer from a file descriptor, reading at least _rlength_ bytes.
+// Throws ReadException if the requested number of bytes cannot be read, with:
+//     errcode = 0   if end of stream (remote end closed)
+//     errcode = errno   if another error occurred
+// Note that EINTR is ignored (i.e. the read will be re-tried).
+static void fillBufferTo(CPBuffer<1024> *buf, int fd, int rlength)
 {
-    int r = buf->fillTo(fd, rlength);
-    if (r == -1) {
-        throw ReadCPException(errno);
-    }
-    else if (r == 0) {
-        throw ReadCPException(0);
+    do {
+        int r = buf->fill_to(fd, rlength);
+        if (r == -1) {
+            if (errno != EINTR) {
+                throw ReadCPException(errno);
+            }
+        }
+        else if (r == 0) {
+            throw ReadCPException(0);
+        }
+        else {
+            return;
+        }
     }
+    while (true);
 }
 
 static const char * describeState(bool stopped)
@@ -52,7 +73,7 @@ static const char * describeVerb(bool stop)
 
 // Wait for a reply packet, skipping over any information packets
 // that are received in the meantime.
-static void wait_for_reply(CPBuffer &rbuffer, int fd)
+static void wait_for_reply(CPBuffer<1024> &rbuffer, int fd)
 {
     fillBufferTo(&rbuffer, fd, 1);
     
@@ -87,12 +108,17 @@ static int write_all(int fd, const void *buf, size_t count)
     return w;
 }
 
+
+static const int START_SERVICE = 1;
+static const int STOP_SERVICE = 2;
+static const int UNPIN_SERVICE = 3;
+
+
 // Entry point.
 int main(int argc, char **argv)
 {
     using namespace std;
     
-    bool do_stop = false;
     bool show_help = argc < 2;
     char *service_name = nullptr;
     
@@ -104,10 +130,7 @@ int main(int argc, char **argv)
     bool wait_for_service = true;
     bool do_pin = false;
     
-    int command = 0;
-    
-    constexpr int START_SERVICE = 1;
-    constexpr int STOP_SERVICE = 2;
+    int command = 0;    
         
     for (int i = 1; i < argc; i++) {
         if (argv[i][0] == '-') {
@@ -128,7 +151,6 @@ int main(int argc, char **argv)
                 do_pin = true;
             }
             else {
-                cerr << "Unrecognized command-line parameter: " << argv[i] << endl;
                 return 1;
             }
         }
@@ -139,6 +161,9 @@ int main(int argc, char **argv)
             else if (strcmp(argv[i], "stop") == 0) {
                 command = STOP_SERVICE;
             }
+            else if (strcmp(argv[i], "unpin") == 0) {
+                command = UNPIN_SERVICE;
+            }
             else {
                 show_help = true;
                 break;
@@ -162,6 +187,7 @@ int main(int argc, char **argv)
         cout << "\nUsage:" << endl;
         cout << "    dinitctl [options] start [options] <service-name> : start and activate service" << endl;
         cout << "    dinitctl [options] stop [options] <service-name>  : stop service and cancel explicit activation" << endl;
+        cout << "    dinitctl [options] unpin <service-name>           : un-pin the service (after a previous pin)" << endl;
         // TODO:
         // cout << "    dinitctl [options] wake <service-name>  : start but don't activate service" << endl;
         
@@ -178,10 +204,11 @@ int main(int argc, char **argv)
         return 1;
     }
     
-    do_stop = (command == STOP_SERVICE);
+    signal(SIGPIPE, SIG_IGN);
     
     control_socket_path = "/dev/dinitctl";
     
+    // Locate control socket
     if (! sys_dinit) {
         char * userhome = getenv("HOME");
         if (userhome == nullptr) {
@@ -204,7 +231,7 @@ int main(int argc, char **argv)
     
     int socknum = socket(AF_UNIX, SOCK_STREAM, 0);
     if (socknum == -1) {
-        perror("socket");
+        perror("dinitctl: socket");
         return 1;
     }
 
@@ -212,7 +239,7 @@ int main(int argc, char **argv)
     uint sockaddr_size = offsetof(struct sockaddr_un, sun_path) + strlen(control_socket_path) + 1;
     name = (struct sockaddr_un *) malloc(sockaddr_size);
     if (name == nullptr) {
-        cerr << "dinit-start: out of memory" << endl;
+        cerr << "dinitctl: Out of memory" << endl;
         return 1;
     }
     
@@ -221,55 +248,45 @@ int main(int argc, char **argv)
     
     int connr = connect(socknum, (struct sockaddr *) name, sockaddr_size);
     if (connr == -1) {
-        perror("connect");
+        perror("dinitctl: connect");
         return 1;
     }
     
     // TODO should start by querying protocol version
     
-    // Build buffer;
-    uint16_t sname_len = strlen(service_name);
-    int bufsize = 3 + sname_len;
-    char * buf = new char[bufsize];
-    
-    buf[0] = DINIT_CP_LOADSERVICE;
-    memcpy(buf + 1, &sname_len, 2);
-    memcpy(buf + 3, service_name, sname_len);
+    if (command == UNPIN_SERVICE) {
+        return unpinService(socknum, service_name, verbose);
+    }
+
+    return startStopService(socknum, service_name, command, do_pin, wait_for_service, verbose);
+}
+
+// Start/stop a service
+static int startStopService(int socknum, const char *service_name, int command, bool do_pin, bool wait_for_service, bool verbose)
+{
+    using namespace std;
+
+    bool do_stop = (command == STOP_SERVICE);
     
-    int r = write_all(socknum, buf, bufsize);
-    delete [] buf;
-    if (r == -1) {
-        perror("write");
+    if (issueLoadService(socknum, service_name)) {
         return 1;
     }
-    
+
     // Now we expect a reply:
     
     try {
-        CPBuffer rbuffer;
+        CPBuffer<1024> rbuffer;
         wait_for_reply(rbuffer, socknum);
         
-        //ServiceState state;
+        ServiceState state;
         //ServiceState target_state;
         handle_t handle;
         
-        if (rbuffer[0] == DINIT_RP_SERVICERECORD) {
-            fillBufferTo(&rbuffer, socknum, 2 + sizeof(handle));
-            rbuffer.extract((char *) &handle, 2, sizeof(handle));
-            //state = static_cast<ServiceState>(rbuffer[1]);
-            //target_state = static_cast<ServiceState>(rbuffer[2 + sizeof(handle)]);
-            rbuffer.consume(3 + sizeof(handle));
-        }
-        else if (rbuffer[0] == DINIT_RP_NOSERVICE) {
-            cerr << "Failed to find/load service." << endl;
-            return 1;
-        }
-        else {
-            cerr << "Protocol error." << endl;
-            return 1;
+        if (checkLoadReply(socknum, rbuffer, &handle, &state) != 0) {
+            return 0;
         }
-        
-        // ServiceState wanted_state = do_stop ? ServiceState::STOPPED : ServiceState::STARTED;
+                
+        ServiceState wanted_state = do_stop ? ServiceState::STOPPED : ServiceState::STARTED;
         int command = do_stop ? DINIT_CP_STOPSERVICE : DINIT_CP_STARTSERVICE;
         
         // Need to issue STOPSERVICE/STARTSERVICE
@@ -277,41 +294,38 @@ int main(int argc, char **argv)
         // start/stop also sets or clears the "explicitly started" flag on the service.
         //if (target_state != wanted_state) {
         {
-            buf = new char[2 + sizeof(handle)];
-            buf[0] = command;
-            buf[1] = do_pin ? 1 : 0;
-            memcpy(buf + 2, &handle, sizeof(handle));
-            r = write_all(socknum, buf, 2 + sizeof(handle));
-            delete buf;
+            int r;
+            
+            {
+                auto buf = new char[2 + sizeof(handle)];
+                unique_ptr<char[]> ubuf(buf);
+                
+                buf[0] = command;
+                buf[1] = do_pin ? 1 : 0;
+                memcpy(buf + 2, &handle, sizeof(handle));
+                r = write_all(socknum, buf, 2 + sizeof(handle));
+            }
             
             if (r == -1) {
-                perror("write");
+                perror("dinitctl: write");
                 return 1;
             }
             
             wait_for_reply(rbuffer, socknum);
             if (rbuffer[0] == DINIT_RP_ALREADYSS) {
+                bool already = (state == wanted_state);
                 if (verbose) {
-                    cout << "Service already " << describeState(do_stop) << "." << endl;
+                    cout << "Service " << (already ? "(already) " : "") << describeState(do_stop) << "." << endl;
                 }
                 return 0; // success!
             }
             if (rbuffer[0] != DINIT_RP_ACK) {
-                cerr << "Protocol error." << endl;
+                cerr << "dinitctl: Protocol error." << endl;
                 return 1;
             }
             rbuffer.consume(1);
         }
         
-        /*
-        if (state == wanted_state) {
-            if (verbose) {
-                cout << "Service already " << describeState(do_stop) << "." << endl;
-            }
-            return 0; // success!
-        }
-        */
-        
         if (! wait_for_service) {
             if (verbose) {
                 cout << "Issued " << describeVerb(do_stop) << " command successfully." << endl;
@@ -332,7 +346,7 @@ int main(int argc, char **argv)
         }
         
         // Wait until service started:
-        r = rbuffer.fillTo(socknum, 2);
+        int r = rbuffer.fill_to(socknum, 2);
         while (r > 0) {
             if (rbuffer[0] >= 100) {
                 int pktlen = (unsigned char) rbuffer[1];
@@ -365,17 +379,17 @@ int main(int argc, char **argv)
                 }
                 
                 rbuffer.consume(pktlen);
-                r = rbuffer.fillTo(socknum, 2);
+                r = rbuffer.fill_to(socknum, 2);
             }
             else {
                 // Not an information packet?
-                cerr << "protocol error" << endl;
+                cerr << "dinitctl: protocol error" << endl;
                 return 1;
             }
         }
         
         if (r == -1) {
-            perror("read");
+            perror("dinitctl: read");
         }
         else {
             cerr << "protocol error (connection closed by server)" << endl;
@@ -383,13 +397,128 @@ int main(int argc, char **argv)
         return 1;
     }
     catch (ReadCPException &exc) {
-        cerr << "control socket read failure or protocol error" << endl;
+        cerr << "dinitctl: control socket read failure or protocol error" << endl;
         return 1;
     }
     catch (std::bad_alloc &exc) {
-        cerr << "out of memory" << endl;
+        cerr << "dinitctl: out of memory" << endl;
         return 1;
     }
     
     return 0;
 }
+
+// Issue a "load service" command (DINIT_CP_LOADSERVICE), without waiting for
+// a response. Returns 1 on failure (with error logged), 0 on success.
+static int issueLoadService(int socknum, const char *service_name)
+{
+    using namespace std;
+    
+    // Build buffer;
+    uint16_t sname_len = strlen(service_name);
+    int bufsize = 3 + sname_len;
+    int r;
+    
+    {
+        // TODO: new: catch exception
+        unique_ptr<char[]> ubuf(new char[bufsize]);
+        auto buf = ubuf.get();
+        
+        buf[0] = DINIT_CP_LOADSERVICE;
+        memcpy(buf + 1, &sname_len, 2);
+        memcpy(buf + 3, service_name, sname_len);
+        
+        r = write_all(socknum, buf, bufsize);
+    }
+    
+    if (r == -1) {
+        perror("dinitctl: write");
+        return 1;
+    }
+    
+    return 0;
+}
+
+// Check that a "load service" reply was received, and that the requested service was found.
+static int checkLoadReply(int socknum, CPBuffer<1024> &rbuffer, handle_t *handle_p, ServiceState *state_p)
+{
+    using namespace std;
+    
+    if (rbuffer[0] == DINIT_RP_SERVICERECORD) {
+        fillBufferTo(&rbuffer, socknum, 2 + sizeof(*handle_p));
+        rbuffer.extract((char *) handle_p, 2, sizeof(*handle_p));
+        if (state_p) *state_p = static_cast<ServiceState>(rbuffer[1]);
+        //target_state = static_cast<ServiceState>(rbuffer[2 + sizeof(handle)]);
+        rbuffer.consume(3 + sizeof(*handle_p));
+        return 0;
+    }
+    else if (rbuffer[0] == DINIT_RP_NOSERVICE) {
+        cerr << "dinitctl: Failed to find/load service." << endl;
+        return 1;
+    }
+    else {
+        cerr << "dinitctl: Protocol error." << endl;
+        return 1;
+    }
+}
+
+static int unpinService(int socknum, const char *service_name, bool verbose)
+{
+    using namespace std;
+    
+    // Build buffer;
+    if (issueLoadService(socknum, service_name) == 1) {
+        return 1;
+    }
+
+    // Now we expect a reply:
+    
+    try {
+        CPBuffer<1024> rbuffer;
+        wait_for_reply(rbuffer, socknum);
+        
+        handle_t handle;
+        
+        if (checkLoadReply(socknum, rbuffer, &handle, nullptr) != 0) {
+            return 1;
+        }
+        
+        // Issue UNPIN command.
+        {
+            int r;
+            
+            {
+                char *buf = new char[1 + sizeof(handle)];
+                unique_ptr<char[]> ubuf(buf);
+                buf[0] = DINIT_CP_UNPINSERVICE;
+                memcpy(buf + 1, &handle, sizeof(handle));
+                r = write_all(socknum, buf, 2 + sizeof(handle));
+            }
+            
+            if (r == -1) {
+                perror("dinitctl: write");
+                return 1;
+            }
+            
+            wait_for_reply(rbuffer, socknum);
+            if (rbuffer[0] != DINIT_RP_ACK) {
+                cerr << "dinitctl: Protocol error." << endl;
+                return 1;
+            }
+            rbuffer.consume(1);
+        }
+    }
+    catch (ReadCPException &exc) {
+        cerr << "dinitctl: Control socket read failure or protocol error" << endl;
+        return 1;
+    }
+    catch (std::bad_alloc &exc) {
+        cerr << "dinitctl: Out of memory" << endl;
+        return 1;
+    }
+    
+    if (verbose) {
+        cout << "Service unpinned." << endl;
+    }
+    return 0;
+}