d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Return SEM_FAILED instead of NULL, though there are same, but the

Browse Source 
SEM_FAILED is more suitable name.
In function, sem_close(), always set errno on error.
This commit is contained in:
davidxu 2010-01-13 08:53:23 +00:00
parent 38619cd970
commit 028e43e0f7
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/libc/gen/sem_new.c
View File

@ -147,13 +147,13 @@ _sem_open(const char *name, int flags, ...)
if (name[0] != '/') {
errno = EINVAL;
return (NULL);
return (SEM_FAILED);
}
name++;
if (flags & ~(O_CREAT|O_EXCL)) {
errno = EINVAL;
return (NULL);
return (SEM_FAILED);
}
_pthread_once(&once, sem_module_init);
@ -275,6 +275,7 @@ _sem_close(sem_t *sem)
return (0);
}
_pthread_mutex_unlock(&sem_llock);
errno = EINVAL;
return (-1);
}