Launchpad has imported 1 comments from the remote bug at
https://bugs.freedesktop.org/show_bug.cgi?id=55678.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2012-10-05T20:21:11+00:00 Alexpigna-dev wrote:

In file src/xlibi18n/lcConv.c the following linked list head is defined

static XlcConverterList conv_list = NULL;

Which is then modified by _XlcSetConverter and get_converter in a non
thread-safe manner. Inside get_converter the list is reorderder to
increase the efficiency of looking up the same element the next time,
but this is especially dangerous since a seemingly read-only method is
actually modifying the data.

Modifying the list in such thread unsafe manner does case the list to
become garbled in some workloads and causes infinite loops when the
get_converter is invoked. The solution I suggest is to add a mutex or
spinlock around accesses to the linked list, I would it myself but I'm
not sure about what is the usual mutex implementation for this project.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libx11/+bug/1062534/comments/0


** Changed in: xlibs
       Status: Unknown => Confirmed

** Changed in: xlibs
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1062534

Title:
  Type unsafe access to internal linked list

To manage notifications about this bug go to:
https://bugs.launchpad.net/xlibs/+bug/1062534/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to