mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-23 06:05:27 +00:00
Fix errno handling for pthreads wrappers
* src/util/threads-pthread.c: pthreads APIs do not set errno, instead the return value is the positive errno. Set errno based on the return value in the wrappers
This commit is contained in:
parent
dd9e9c3b54
commit
15a9144609
@ -35,8 +35,9 @@ void virThreadOnExit(void)
|
||||
|
||||
int virMutexInit(virMutexPtr m)
|
||||
{
|
||||
if (pthread_mutex_init(&m->lock, NULL) != 0) {
|
||||
errno = EINVAL;
|
||||
int ret;
|
||||
if ((ret = pthread_mutex_init(&m->lock, NULL)) != 0) {
|
||||
errno = ret;
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
@ -57,11 +58,11 @@ void virMutexUnlock(virMutexPtr m)
|
||||
}
|
||||
|
||||
|
||||
|
||||
int virCondInit(virCondPtr c)
|
||||
{
|
||||
if (pthread_cond_init(&c->cond, NULL) != 0) {
|
||||
errno = EINVAL;
|
||||
int ret;
|
||||
if ((ret = pthread_cond_init(&c->cond, NULL)) != 0) {
|
||||
errno = ret;
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
@ -69,8 +70,9 @@ int virCondInit(virCondPtr c)
|
||||
|
||||
int virCondDestroy(virCondPtr c)
|
||||
{
|
||||
if (pthread_cond_destroy(&c->cond) != 0) {
|
||||
errno = EINVAL;
|
||||
int ret;
|
||||
if ((ret = pthread_cond_destroy(&c->cond)) != 0) {
|
||||
errno = ret;
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
@ -78,8 +80,9 @@ int virCondDestroy(virCondPtr c)
|
||||
|
||||
int virCondWait(virCondPtr c, virMutexPtr m)
|
||||
{
|
||||
if (pthread_cond_wait(&c->cond, &m->lock) != 0) {
|
||||
errno = EINVAL;
|
||||
int ret;
|
||||
if ((ret = pthread_cond_wait(&c->cond, &m->lock)) != 0) {
|
||||
errno = ret;
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
@ -99,8 +102,9 @@ void virCondBroadcast(virCondPtr c)
|
||||
int virThreadLocalInit(virThreadLocalPtr l,
|
||||
virThreadLocalCleanup c)
|
||||
{
|
||||
if (pthread_key_create(&l->key, c) != 0) {
|
||||
errno = EINVAL;
|
||||
int ret;
|
||||
if ((ret = pthread_key_create(&l->key, c)) != 0) {
|
||||
errno = ret;
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
|
Loading…
Reference in New Issue
Block a user