Fix: sessiond: return _OK on _SET_SESSION_SHM_PATH command success
authorFrancis Deslauriers <francis.deslauriers@efficios.com>
Thu, 11 Mar 2021 16:06:53 +0000 (11:06 -0500)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Thu, 11 Mar 2021 20:31:53 +0000 (15:31 -0500)
commit7e397c55fae384e3218be22e13996d7989611f25
tree5879dc2d9cc79fc78174764e5938f4094fb2ed22
parentf2e97f594f0c730f933b5f9840938d878b13e26b
Fix: sessiond: return _OK on _SET_SESSION_SHM_PATH command success

Issue
=====
When running the `test_crash` tests, I witness the following warning:
  Warning: Command returned an invalid status code, returning unknown error: command type = 40, ret = 0

The `cmd_set_session_shm_path()` function is returning 0 on success
instead of the expected LTTNG_OK which leads to this warning being
printed.

Fix
===
Return LTTNG_OK.

Side note
=========
I added the string version of the command name in the warning printing
for easier debugging.

Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I79fe41afd49a1c701e760e415b72aabdc9b25f6c
src/bin/lttng-sessiond/client.c
src/bin/lttng-sessiond/cmd.c
This page took 0.032096 seconds and 4 git commands to generate.