lttng-ust:stable-2.12

Last commit made on 2020-03-27
Get this branch:
git clone -b stable-2.12 https://git.launchpad.net/lttng-ust

Branch merges

Branch information

Name:
stable-2.12
Repository:
lp:lttng-ust

Recent commits

965cd4a... by Mathieu Desnoyers on 2020-03-27

Version 2.12.0-rc3

Signed-off-by: Mathieu Desnoyers <email address hidden>

628f0e4... by Jonathan Rajotte on 2020-03-09

Fix: v.u.d might be uninitialized

lttng-filter-interpreter.c:301:17: warning: ‘v.u.d’ may be used uninitialized in this function [-Wmaybe-uninitialized]
   ptr->u.d = v.u.d;

Indeed it seems that the value is never fetched if we compare to other
call sites and the dynamic call site further below.

Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>
Change-Id: I542d6db7514a2c39afe3613228577f34f194d672

b52a910... by Mathieu Desnoyers on 2020-03-04

Version 2.12.0-rc2

Signed-off-by: Mathieu Desnoyers <email address hidden>

18ef3e5... by Jonathan Rajotte on 2020-03-02

Fix: set FD_CLOEXEC on incoming FDs.

The stream shm FDs are allocated by the consumer process, and then
passed to the applications over unix sockets. When opening those
file descriptors on reception, the FD_CLOEXEC flag is not set.

In a fork + exec scenario, parent process streams shm FDs and channel
wake FDs are present in the resulting child process.

Set FD_CLOEXEC on reception (ustcomm_recv_fds_unix_sock) to
prevent such scenario.

Change-Id: Id58077b272be9c1ab239846639ffd8103b3d50f1
Signed-off-by: Jonathan Rajotte <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>

38b48c9... by Mathieu Desnoyers on 2020-02-18

Fix: tracepoint.h: Disable address sanitizer on pointer array section variables

The tracepoint header declares pointer global variables meant to be
placed contiguously within the __tracepoints_ptrs section, and then used
as an array of pointers when loading an executable or shared object.

Clang Address Sanitizer adds redzones around each variable, thus leading to
detection of a global buffer overflow.

Those redzones should not be placed within this section, because it
defeats its purpose. Therefore, teach asan not to add redzones
around those variables with an attribute.

Note that there does not appear to be any issue with gcc (tested with
gcc-8 with address sanitization enabled), and gcc ignores the
no_sanitize_address attribute when applied to a global variable.

Fixes: #1238

Signed-off-by: Mathieu Desnoyers <email address hidden>

ec9e040... by Mathieu Desnoyers on 2020-02-18

Fix: jhash.h: remove out-of-bound reads

jhash.h implements "special" code for valgrind because it reads memory
out-of-bound (and then applies a mask) when reading strings.

Considering that lttng-ust does not use jhash.h in a fast-path, remove
this "optimization" and use the verifiable VALGRIND code instead. This
fixes an ASan splat.

Fixes: #1238

Signed-off-by: Mathieu Desnoyers <email address hidden>

e5ea8cc... by Francis Deslauriers <email address hidden> on 2020-02-17

Cleanup: remove trailing white spaces across project

Signed-off-by: Francis Deslauriers <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>
Change-Id: I15202338465ee56d33316cbc632d9e3bf44ee31e

b2a050e... by Francis Deslauriers <email address hidden> on 2019-11-26

Fix: lttng-ust-comm.c: return number of fd rather size of array

There are two conflicting comments for this function. One says it
returns the size of the received data and the other says it returns the
number of fd received.

It's more useful to receive the number of fd.

Signed-off-by: Francis Deslauriers <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>
Change-Id: I74084b461d396c3e623fa55100e6dd7e59dbea83

747ecf4... by Francis Deslauriers <email address hidden> on 2019-12-06

liblttng-ust: exit loop early on event enabler match

Signed-off-by: Francis Deslauriers <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>
Change-Id: I0fa3215f7cd6a2d32ac00d66cf5fc184abd14612

1db47a7... by Francis Deslauriers <email address hidden> on 2019-11-27

Add git-review config

Add .gitreview for contributors wishing to use gerrit for patch
reviews.

Signed-off-by: Francis Deslauriers <email address hidden>
Signed-off-by: Mathieu Desnoyers <email address hidden>
Change-Id: I3ba853d0d68a660ac5d2d24c757e0f28b634977c