Fix: remove assert on fd in the read/write layer

It is possible that an invalid fd is passed to read or write. This can
happen for instance if the endpoint of the transport (ex: relayd) dies
out while actively trying to send data. It is OK to let an invalid fd
where the syscall will return the right value along with errno being
populated with the corresponding code.

Acked-by: Julien Desfossez <julien.desfossez@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>

diff --git a/src/common/readwrite.c b/src/common/readwrite.c
index 0098f759b2d57dc156088126d457a86226517e65..7b8460962eda5ee7e917be20cce7c47be0a56ea4 100644 (file)
--- a/src/common/readwrite.c
+++ b/src/common/readwrite.c
@@ -34,7 +34,6 @@ ssize_t lttng_read(int fd, void *buf, size_t count)
        size_t i = 0;
        ssize_t ret;
 
-       assert(fd >= 0);
        assert(buf);
 
        do {
@@ -64,7 +63,6 @@ ssize_t lttng_write(int fd, const void *buf, size_t count)
        size_t i = 0;
        ssize_t ret;
 
-       assert(fd >= 0);
        assert(buf);
 
        do {