Fix utils::writeToFd return type
authordisconnect3d <dominik.b.czarnota@gmail.com>
Sat, 24 Nov 2018 15:23:45 +0000 (16:23 +0100)
committerdisconnect3d <dominik.b.czarnota@gmail.com>
Sat, 24 Nov 2018 15:40:30 +0000 (16:40 +0100)
The `writeToFd` function in `util.cc` returns `ssize_t` but the only
returned values are either `false` or `true`.

```
ssize_t writeToFd(int fd, const void* buf, size_t len) {

(...) return false;

(...) return true;
```

subproc.cc
util.cc

index 339b5bf..f10a5e4 100644 (file)
@@ -383,8 +383,7 @@ static bool initParent(nsjconf_t* nsjconf, pid_t pid, int pipefd) {
                LOG_E("Couldn't initialize user namespace for pid %d", pid);
                return false;
        }
-       if (util::writeToFd(pipefd, &kSubprocDoneChar, sizeof(kSubprocDoneChar)) !=
-           sizeof(kSubprocDoneChar)) {
+       if (!util::writeToFd(pipefd, &kSubprocDoneChar, sizeof(kSubprocDoneChar))) {
                LOG_E("Couldn't signal the new process via a socketpair");
                return false;
        }
diff --git a/util.cc b/util.cc
index ddf39b8..e271e9d 100644 (file)
--- a/util.cc
+++ b/util.cc
@@ -75,7 +75,7 @@ ssize_t readFromFile(const char* fname, void* buf, size_t len) {
        return ret;
 }
 
-ssize_t writeToFd(int fd, const void* buf, size_t len) {
+bool writeToFd(int fd, const void* buf, size_t len) {
        const uint8_t* charbuf = (const uint8_t*)buf;
 
        size_t writtenSz = 0;