lttng-ctl: fix: lttng_data_pending confuses communication status
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 11 Sep 2019 16:18:15 +0000 (12:18 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 11 Sep 2019 16:18:30 +0000 (12:18 -0400)
commite848d36d1b0e7cbe1800e195fb00eaff6bce4d5e
treed8604350d794e9522368a16514af66790439dd83
parent4147107d9646011a1cdb7166a9810065927af99c
lttng-ctl: fix: lttng_data_pending confuses communication status

lttng_ctl_ask_sessiond can return a positive value even though it
failed to receive the variable length payload of a session message
reply. In this case, lttng_ctl_ask_sessiond ends up calling into
lttng_ctl_ask_sessiond_fds_varlen() which will return the (negated)
error code returned by the session daemon if it was not LTTNG_OK.

The peer could return anything here, which lttng_data_pending will end
up interpreting as the length of the variable data that was received.

In this case, if the sessiond returns '-1', '1' will be returned to
lttng_data_pending, which it will interpret as being the length of the
'data_pending' byte flag. It will then dereference 'pending', which is
NULL, and (most likely) crash.

Check for NULL on top of checking for the return code. This
communication layer needs love as much as it needs a bulldozer.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/lib/lttng-ctl/lttng-ctl.c
This page took 0.032345 seconds and 4 git commands to generate.