Incorrect checking of flags could result in uninitialized file descriptor being used.
Signed-off-by: Liam Merwick <liam.merw...@oracle.com> Reviewed-by: Darren Kenny <darren.ke...@oracle.com> Reviewed-by: Mark Kanda <mark.ka...@oracle.com> --- io/channel-command.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/io/channel-command.c b/io/channel-command.c index 3e7eb17eff54..38deb687da21 100644 --- a/io/channel-command.c +++ b/io/channel-command.c @@ -59,10 +59,10 @@ qio_channel_command_new_spawn(const char *const argv[], flags = flags & O_ACCMODE; - if (flags == O_RDONLY) { + if ((flags & O_RDONLY) == O_RDONLY) { stdinnull = true; } - if (flags == O_WRONLY) { + if ((flags & O_WRONLY) == O_WRONLY) { stdoutnull = true; } -- 1.8.3.1