lttng-tools:stable-2.8

Last commit made on 2017-08-03
Get this branch:
git clone -b stable-2.8 https://git.launchpad.net/lttng-tools

Branch merges

Branch information

Name:
stable-2.8
Repository:
lp:lttng-tools

Recent commits

73d0ff3... by =?utf-8?q?J=C3=A9r=C3=A9mie_Galarneau?= <email address hidden>

Update version to v2.8.8

Signed-off-by: Jérémie Galarneau <email address hidden>

b0c07e4... by Mathieu Desnoyers

Fix: uninitialized return value on error path

Found by Coverity:

*** CID 1378810: Uninitialized variables (UNINIT)
/src/bin/lttng-sessiond/context.c: 73 in add_kctx_all_channels()

Signed-off-by: Mathieu Desnoyers <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

bf82eb3... by =?utf-8?q?J=C3=A9r=C3=A9mie_Galarneau?= <email address hidden>

Fix: ensure kernel context is in a list before trying to delete it

Signed-off-by: Jérémie Galarneau <email address hidden>

f712b08... by =?utf-8?q?J=C3=A9r=C3=A9mie_Galarneau?= <email address hidden>

Fix: ambiguous ownership of kernel context by multiple channels

A kernel context, when added to multiple channels, must be copied
before being added to individual channels. The current code
adds the same ltt_kernel_context structure to multiple kernel
channels which introduces a conceptual ambiguity in the ownership
of the context object.

Concretely, creating multiple kernel channels and adding a context
to all of them (by not specifying a channel name) causes the context
to be added to each channels' list of contexts, overwritting the
context's list node, and causing the channel context lists to become
corrupted. This results in crashes being observed during the
destruction of the session.

Signed-off-by: Jérémie Galarneau <email address hidden>

51ae18c... by Jonathan Rajotte

Fix: ret is never used on error_open code path

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

6f386b4... by Jonathan Rajotte

Fix: use error code path instead of break when errors happen before execl

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

02e93b8... by Jonathan Rajotte

Fix: wrong variable assignment on error

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

aa41a0f... by Jonathan Rajotte

Fix: missing error handling in use of print_tabs()

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

70ac723... by Jonathan Rajotte

Fix: ret is used instead or err to set an error code

Use err instead of ret. ret is never used for error reporting under
error label.

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>

e48a0a8... by Jonathan Rajotte

Fix: report error using fd instead of ret

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Jérémie Galarneau <email address hidden>