io: Initialize pipe file descriptors to -1.

If we abort setup due to I/O initialization failure, we call PIPE_CLOSE
which will close the pipes if the file descriptors are not -1. However,
we were not initializing the file descriptors to -1, which left them at
0 after calloc(), leading us to attempt to close fd 0.

We now initialize all the pipe file descriptors to -1 properly in all
I/O modules.

LBBS-142 #close

Author: InterLinked1

io/io_compress.c

@@ -214,6 +214,10 @@ static int setup(int *rfd, int *wfd, enum bbs_io_transform_dir dir, void **restr
 		return -1;
 	}
 
+	/* In case we abort, make sure the file descriptors are valid */
+	z->rpfd[0] = z->rpfd[1] = -1;
+	z->wpfd[0] = z->wpfd[1] = -1;
+
 	z->compressor = &z->compressor_s;
 	z->decompressor = &z->decompressor_s;
 	z->level = DEFAULT_COMPRESSION_LEVEL;

io/io_log.c

@@ -186,6 +186,10 @@ static int setup(int *rfd, int *wfd, enum bbs_io_transform_dir dir, void **restr
 		return -1;
 	}
 
+	/* In case we abort, make sure the file descriptors are valid */
+	l->rpfd[0] = l->rpfd[1] = -1;
+	l->wpfd[0] = l->wpfd[1] = -1;
+
 	l->fp = bbs_mkftemp(template, 0600);
 	if (!l->fp) {
 		goto fail;

io/io_tls.c

@@ -302,6 +302,11 @@ static struct tls_client *ssl_launch(SSL *ssl, int fd, int *rfd, int *wfd, int c
 	if (ALLOC_FAILURE(t)) {
 		return NULL;
 	}
+
+	/* In case we abort, make sure the file descriptors are valid */
+	t->readpipe[0] = t->readpipe[1] = -1;
+	t->writepipe[0] = t->writepipe[1] = -1;
+
 	t->ssl = ssl;
 	t->fd = fd;
 	if (pipe(t->readpipe)) {