diff options
author | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-07-24 00:16:06 +0200 |
---|---|---|
committer | Eelco Dolstra <eelco.dolstra@logicblox.com> | 2014-07-24 00:16:06 +0200 |
commit | 1eb0af7ed563c6d77e1b8e36e81ba6ede7e49f5a (patch) | |
tree | b1f7c97ac1aa8940271c41951ac5e648e43cb7e8 | |
parent | d3c61d83be254790101a67904b37e5439e98b4b2 (diff) |
Add some assertions
-rw-r--r-- | src/libutil/monitor-fd.hh | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/libutil/monitor-fd.hh b/src/libutil/monitor-fd.hh index d8db6508a..ebfc230f9 100644 --- a/src/libutil/monitor-fd.hh +++ b/src/libutil/monitor-fd.hh @@ -1,6 +1,7 @@ #pragma once #include <thread> +#include <atomic> #include <poll.h> #include <sys/types.h> @@ -14,10 +15,12 @@ class MonitorFdHup { private: std::thread thread; + std::atomic_bool quit; public: MonitorFdHup(int fd) { + quit = false; thread = std::thread([&]() { /* Wait indefinitely until a POLLHUP occurs. */ struct pollfd fds[1]; @@ -25,8 +28,10 @@ public: fds[0].events = 0; if (poll(fds, 1, -1) == -1) { if (errno != EINTR) abort(); // can't happen + assert(quit); return; // destructor is asking us to exit } + assert(fds[0].revents & POLLHUP); /* We got POLLHUP, so send an INT signal to the main thread. */ kill(getpid(), SIGINT); }); @@ -34,6 +39,7 @@ public: ~MonitorFdHup() { + quit = true; pthread_kill(thread.native_handle(), SIGINT); thread.join(); } |