Fix: cmd_rotate_session() returns unexpected error codes
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 14 Nov 2018 22:30:17 +0000 (17:30 -0500)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 16 Nov 2018 20:48:07 +0000 (15:48 -0500)
commitd5a1b7aa06b4c924b1cd30623758343c74ecab5c
tree9728a387ef1b8ae136e9b046581b2f50ad0d8a0a
parent61539f9685f7786bc6fe8d23a7c1cef4fa716b7c
Fix: cmd_rotate_session() returns unexpected error codes

Certain paths in cmd_rotate_session() result in its return value not
obeying the convention for commands: return LTTNG_OK on success, and a
negative LTTNG_ERR_* code on error.

This patch separates the use of 'int ret' from a separate 'enum
lttng_error_code' value to ensure 'ret' values never bubble-up to the
caller.

Note that this patch assumes that ust_app_rotate_session() returns an
lttng_error_code, which is not the case right now (upcoming patch).

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
src/bin/lttng-sessiond/cmd.c
src/bin/lttng-sessiond/kernel.c
src/bin/lttng-sessiond/kernel.h
src/bin/lttng-sessiond/ust-app.h
This page took 0.03665 seconds and 5 git commands to generate.