Project

General

Profile

Bug #737

Unsynchronized access to enabledEventList

Added by Jérémie Galarneau about 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Target version:
Start date:
02/24/2014
Due date:
% Done:

0%

Estimated time:

Description

See this thread for explanation and a tentative patch.

http://lists.lttng.org/pipermail/lttng-dev/2014-February/022433.html

#1

Updated by Mathieu Desnoyers about 6 years ago

  • Status changed from New to Resolved

Fixed by commit:

commit e614d916bcf4093bbf98d302b9e37ed990dc7304
Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date:   Tue Feb 25 16:32:05 2014 -0500

    Fix: Unsynchronized access in LTTngTCPSessiondClient

    enabledEventList is shared between the LTTngThread and eventTimer
    threads but is not synchronized.

    This patch changes enabledEventList's type from an ArrayList to a
    synchronized HashSet.

    Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
    Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Also available in: Atom PDF