tree 372d705f39f0e482a61815bd25b40b34124b88f3
parent 4a0eacbc6f4268c29c1c8d13a89aa42f689bb91c
author Chris Dickens <christopher.a.dickens@gmail.com> 1441877960 -0700
committer Chris Dickens <christopher.a.dickens@gmail.com> 1443502290 -0700

core: Prevent attempts to recursively hande events

Prior to this commit, it was possible to call certain functions from
within a hotplug or transfer completion callback that would in turn
attempt to handle events (e.g. any of the sync transfer APIs). This
is dangerous because certain events may cause the nested calls to
free memory that is currently in use by the previous calls.

This implementation uses thread-local storage to store a key within
the context that is set to a non-NULL value whenever event handling
is occurring. This allows us to detect when dangerous calls are made
from within event handling context. Such calls will return an error
code of LIBUSB_ERROR_BUSY.

Note that this implementation was chosen because of its portability.
It is supported on all platforms that libusb supports.

Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>