> if (fds[i][0] > 0) {
> SAFE_CLOSE(fds[i][0]);
> SAFE_CLOSE(fds[i][1]);
> }
It's worth to change the statement to check for fd != -1.
Otherwise:
Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>
--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com