Fix insecure library loading (Debian Bug #598309, CVE-2010-3386) (v2)

Thanks to Raphael Geissert for spotting this, and to Jon Bernard for forwarding
the information to us.

Changelog since v1:
Apply comments from Nils.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Acked-by: Nils Carlson <nils.carlson@ericsson.com>
CC: Raphael Geissert <geissert@debian.org>
CC: Jon Bernard <jbernard@debian.org>

Fix a zmalloc bug

sizeof(void *) was allocated instead of sizeof(struct probe).

Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>

New naming convention for internal macros

This is the first patch introducing the new naming convention
for the UST tracer. To prevent namespace pollution, _ust_ prefix
is added to internal macros which are visible in the global header.

For this patch, only the container_of macro is renamed and the
redundant definition in kcompat/simple.h is removed.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Nils Carlson <nils.carlson@ericsson.com>

Re-write ustcomm parts of UST v2

Changes since v1:

Updated after comments from David Goulet and resulting insights.
* Add a continue after a failed accept
* Fix some malloc issues
* Fix some coding style in the patch
* make del_named_sock free the memory, even if all else fails.

Notice: valgrind test-case currently broken. Needs an exception.

Description:

This is a very big patch, and so it requires a bit of explaining.

This patch is a step on the way of accomplishing serveral goals I have in this
area:

1. Use enums for commands and eliminate text-based commands. This does not mean
   that we will stop processing strings for trace/channel and marker names;
   just that the long series of if statements with token and string matching
   will be replaced with a switch statement. To this end I have created a
   ustcomm_header struct that contains the length of the data-field and some
   other fields. This allows us to first receive the header, allocate memory
   for the data and then receive the data; eliminating all scanning of messages.

2. Reduce the complexity of the implementation. To put it simply, I don't like
   callbacks. They reduce transparency and make it difficult to follow the
   flow of the code; so I have eliminated multipoll replacing it with a normal
   epoll. I have also replaced almost all the different server, connection and
   source structs with one, called ustcomm_sock.

3. Make ustd scale better. Currently ustd scales terribly. We allocate one
   thread per-cpu per-channel per-process, five applications each with three
   channels on a four cpu machine leads to 5*3*4=60 threads. Part of the reason
   for this multitude of threads was that we used a ustcomm_request call
   (consisting of a send and a receive) to wait for a subbuffer to be written.
   The sequence for a subbuffer to be written was as follows:

      Ustd calls send with a 'get_subbuffer' command, and then recv in one of
      the threads and hangs on the recv on the socket.

      Upon filling the subbuffer the traced app writes '1' to a pipe.

      The ust_thread inside the app which was listening to the other end of the
      pipe wakes up when the '1' is written. The callback from multipoll calls
      a send which sends a reply to the ustd thread over the socket.

      The ustd thread wakes up and reads the message, continuing along in its
      execution.

   I replace this with a bit of a different mechanism, which should allow us
   to eventually reduce the number of threads to one per cpu:

      Ustd requests a buffer_fd which causes the ustd_thread inside the app
      to send the file-descriptor for the read en of the pipe to ustd.

      The ustd thread now does a read on the pipe, halting its execution until
      the app fills the subbuffer and writes '1' to the pipe, waking up the ustd
      thread.

      Ustd now makes the 'get_subbuffer' call which the ust_thread inside the
      app responds to with information about the subbuffer. Writes it and then
      goes back to the read call, hanging on the pipe.

   So we are still stuck on the multitude of threads, but we are in much better
   position to move forward. Replacing the read with an epoll statement and then
   pointing the epoll event data at the buffer struct containing the current
   buffer to whitch the pipe belongs should be relatively easy. We can then
   instead of spawning a new thread for each buffer just allocate the
   buffer_info struct and assign it to one of the per-cpu threads in ustd to
   poll on.

4. Replace poll with epoll which scales better, especially for
   events << (nr of fds). This is complete.

5. Allow UST to handle arbitrarily long unix socket names. This is done by
   carefull allocation of the socketaddr_un struct with a dynamic length.
   Truncating is ugly and dangerous.

There is a lot of work still left to be done. This is only the first of a
number of patches that I expect in this area. If someone feels like tackling
ustd head on to reduce the number of threads that would be great.

I have kept Pierre-Marc's form of error handling for the I/O wrapping functions
because I want to propagate return codes up to the apps that are using them
so they can close file-descriptors and free associated resources. If somebody
knows of a better approach please make yourself heard.

Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>
Acked-by: David Goulet <david.goulet@polymtl.ca>

Fix a strange bash anomaly in test_functions.sh

If anybody has time time to investigate this that would be great.
Unforunately, I don't.

Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>

fix sscanf format string v3

Signed-off-by: Douglas Santos <douglas.santos@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

tracectl cleanup v3

kernel style-> if, else, for, while
also remove a trailing space

Signed-off-by: Douglas Santos <douglas.santos@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Remove unused code

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Nils Carlson <nils.carlson@ericsson.com>

Syntax cleanup

Replace spaces by tabs and remove trailing white spaces

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Nils Carlson <nils.carlson@ericsson.com>

Change code for consistency

Use the kcompat stringify header for double level
stringification for markers section.

Also use the GET_MARKER macro for uniformity across the code.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Nils Carlson <nils.carlson@ericsson.com>

null check for disable_marker

Signed-off-by: Douglas Santos <douglas.santos@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Correct error handling and add error message in ustcmd.c

Signed-off-by: Nils Carlson <nils.carlson@ericsson.com>
Acked-by: Douglas Santos <douglas.santos@polymtl.ca>

Add functions and command line for listing trace_events v3

Changes from version 2: Fix a comment.

Changes from version 1: Fix silly things per Davids comments,
realise that we are looping over pids and therefore need to set
all pointers to NULL and all counters to zero each time.

Add trace_event library initialisation

Add library initialisation with dummy trace_event in order
to create sections.

Add trace_event structs

Add trace_event structs, registration functions and trace_event
iterator and locking functions.

Set the exit status of ustctl main function v3

Each possible command checks the return value and, on error,
print on stderr and set the retval. This makes ustctl return
an EXIT_FAILURE on any command that fails.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix missing parenthese around logic comparison

Error introduce in commit number 9dec086e052cf7f583a3afaa3aab48a6de8d38ac.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix DBG to ERR for data format mismatch

Signed-off-by: David Goulet <david.goulet@polymtl.ca>
Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Fix return value handling for libustcmd

Return value was not checked correctly so this was
triggering a free() on an invalid pointer causing
ustctl to fail badly.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Change malloc to zmalloc for libust

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix missing data pointer for trace_mark_tp v2

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix retval value assignment on error

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix asprintf and scanf ignoring return value

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Fix a broken test makefile

The makefile was missing the header file in the sources list and
so was failing make distcheck.

Update test-case for tracepoints and add a new test-case

Update the test-case for probes to handle a new data pointer
in the probe function. Also add a new test-case that stress-tests
probe registration and unregistration.

Add a data pointer to all tracepoint probe callbacks

RCU now dereferences an array of probe structs containing data pointers and
function pointers, and passes the data pointer as the first argument to the
probe.

Due to this change void prototypes are no longer allowed, which has incurred
changes to libust-initializer.c .

This patch also fixes a problem where non-existent tracepoints are present
in the __tracepoints section and with a name pointer pointing at NULL.

Add a simple test case for TRACE_EVENT v2

The test case is useful for verifying compilation of the simplest
possible TRACE_EVENT.

Changes since version 1:
Added copyright notices
updated run with UST_AUTOCOLLECT per Davids comment.

Import TRACE_EVENT macro from the kernel v3

This is a slightly modified version of the TRACE_EVENT
macro from the kernel, TP_printf has replaced TP_printk.

The TRACE_EVENT macro expansion is currently a dummy which
only expands to a printf probe, connected in a constructor.

Changes since version 2:
Added re-licensing information
Macro naming is more consisten (always contains trace)

Changes since version 1:
Added new headers to include/Makefile.am
Added major copyright holders and license information to headers.

Add LGPLv2.1 license text

update ustctl man page

Final updates for release 0.7

Changes malloc to zmalloc

For libustcomm, libustd, ustctl and ustd.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Removes useless ___cacheline_aligned

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Updates Changelog for UST version 0.6

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

UST strncpy assert fix

fix off by one in strncpy and write asserts(). Shows only when the event
reserved fits the exact end of buffer.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

properly install libustd and libustcmd

don't install libustcomm and libustsnprintf

These internal libraries should not be exposed to users and were
only installed by mistake.

remove dependency on ustcomm.h where possible

ustcomm structures are replaced with pointers so the compiler doesn't
have to know the contents of the structures. This is done so ustcmd.h
and libustd.h can be installed on a system without needing to install
ustcomm.h.

remove unused variables

Final updates for release 0.6

serialize string input robustness

Make sure we handle concurrently modified input strings gracefully.

Adds ust_buffers_strncpy interface. It fixes up the string if it does not fills
exactly the space allocated. Ensures the string ends with \0.

Removes unused "_ust_buffers_write" at the same time.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

UST synchronization fix

Missing smp_wmb() at subbuffer switch.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

fix invalid array access

reported by Damien Cottier

ustctl manpage: document force-switch

add a command to force subbuffer switch

remove ltt_relay_buffer_flush function

All it did was call ltt_force_switch and wake up consumers, when
ltt_force_switch already wakes up the consumers.

ustctl manpage: document get/set-sock-path

add get/set commands for daemon socket path

create libustd from ustd

This makes libustd a library containing the core functionality of
the daemon, and ustd a simple client application using this library,
interacting with it via a set of callbacks supplied to it.

add paranoid check before listener thread destruction

In case a pthread error prevented its creation.

define ____cacheline_aligned

Fix directory creation mode. Unless umask is set to 0 the default umask of the system, commonly 022, will be used and thus it becomes impossible to get the 777 mode.

UST marker.h

Modification nécessaire pour faire des custom probes avec trace_mark_tp.
Sous la recommandation de Michel et Mathieu, il était mieux d'utiliser
cela pour les tests de profilage que des trace_mark ou des tracepoints.

Ceci étant dit, les résultats avec un test de profilage que j'ai monté,
contenant 3 tracepoints, le average perevent était de ~65ns.

Maintenant, résultats préliminaires avec les custom probes sur le même
test suite que les trace_mark et tracepoints :

Average time per event                  : 0.0000000159 (sec), 0.00002
(msec), 0.01587 (usec), 15.86718 (nsec)

Best time [RUN 5]                       : 0.0000000154 (sec), 0.00002
(msec), 0.01536 (usec), 15.35519 (nsec)
Worst time [RUN 6]                      : 0.0000000166 (sec), 0.00002
(msec), 0.01659 (usec), 16.59236 (nsec)
Standard Deviation                      : 0.0000000004 (sec), 0.00000
(msec), 0.00038 (usec), 0.38116 (nsec)

Quand même! ;)

Bref, je continue mais si tu veux commenter ou proposer des tests, fait
moi signe.

A plus
David

From e1fc903e0cedfc21895b2ad952bb6127d0b3b853 Mon Sep 17 00:00:00 2001
From: David Goulet <david.goulet@polymtl.ca>
Date: Wed, 23 Jun 2010 17:04:45 -0400
Subject: [PATCH] Fixed __trace_mark_tp_counter with the usage of
 DEFINE_MARKER_TP macro. Down to 5 arguments instead of 7
 and added registers struct missing. Now, custom probes
 can be used within UST with trace_mark_tp function

Removing NO_IPI ifdef for memory barrier. The smp_wmb should be used all the time here.

Signed-off-by: David Goulet <david.goulet@polymtl.ca>

Final updates for release 0.5

update doc for next release

fix regression of GDB support

Address of marker was not reported correctly. Fix by Pedro Alves.

tracectl.c: add description comment

update READMEs

add info about what dirs/files are doing

add environment variables to set subbuffer options

UST_SUBBUF_SIZE and UST_SUBBUF_NUM control respectively the size and
number of subbuffers. Options -S and -N in the usttrace script also
control these parameters.

update README

make RUNLTTV configurable in trace_matches

update -W help in the usttrace man page

Changes testcases in runtests to use TAP

TAP - Test Anything Protocol is an easy to parse test protocol.

update manual

update README

update manual

block all signals in listener thread

Not doing this can result in a situation where all the other application
threads have a signal blocked, and the listener thread, not having it blocked,
lets the signal handler execute. Therefore, in order to be as transparent as
possible to the application, we have to block signals.

documentation update

Replace gettimeofday() with clock_gettime(CLOCK_MONOTONIC,...)

Contributed by Nils Carlson.

remove mutex_lock, mutex_unlock macros

test for two markers on the same line

usttrace man page update

Test for a dynamically linked library

Correction of dynamic library marked test

runtests: remove unimplemented test

add average script

add missing ptime for benchmark

update tracepoints example with api updates

Update benchmark

fix power of two computation

fix warning

fix bootstrapping

change to silent compilation

Add support for ppc hw tb clock, remove kernelcompat.h

add a .gitignore for java

add custom probes support and update tracepoints

remove some unused code

Port to PowerPC 32

jhash: add cast

compile with -fno-strict-aliasing

add support for channel overwrite and non-collection

update TODO

Add manpages for ustd, ustctl, and usttrace

testsuite: update for libmallocwrap rename to libustinstr-malloc

benchmark: fix various problems

benchmark: fix makefile

benchmark tool

remove unused consumer_stack

fix pthread error handling