[libvirt] [PATCH] win32: Properly handle TlsGetValue returning NULL
Eric Blake
eblake at redhat.com
Sun Apr 22 02:32:58 UTC 2012
On 04/21/2012 11:11 AM, Matthias Bolte wrote:
> virThreadSelf tries to access the virThreadPtr stored in TLS for the
> current thread via TlsGetValue. When virThreadSelf is called on a thread
> that was not created via virThreadCreate (e.g. the main thread) then
> TlsGetValue returns NULL as TlsAlloc initializes TLS slots to NULL.
>
> virThreadSelf can be called on the main thread via this call chain from
> virsh
>
> vshDeinit
> virEventAddTimeout
> virEventPollAddTimeout
> virEventPollInterruptLocked
> virThreadIsSelf
>
> triggering a segfault as virThreadSelf unconditionally dereferences the
> return value of TlsGetValue.
>
> Fix this by making virThreadSelf check the TLS slot value for NULL and
> setting the given virThreadPtr accordingly.
>
> Reported by Marcel Müller.
> ---
> src/util/threads-win32.c | 11 +++++++++--
> 1 files changed, 9 insertions(+), 2 deletions(-)
ACK.
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120421/3d013bb2/attachment-0001.sig>
More information about the libvir-list
mailing list