michaelh/zephyr
1
0
Fork 0
Code Releases Activity

lib: posix: Fix unchecked return value in rwlock

Browse source 
Fix 'sys_sem_xxx' functions unchecked return value
scanned by Coverity.

Signed-off-by: James Roy <rruuaanng@outlook.com>
This commit is contained in:
James Roy 2024-11-29 22:55:39 +08:00 • committed by Benjamin Cabé
commit c875582443
1 changed files with 8 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

16
lib/posix/options/rwlock.c
View file

@ -331,15 +331,15 @@ int pthread_rwlock_unlock(pthread_rwlock_t *rwlock)
if (k_current_get() == rwl->wr_owner) { if (k_current_get() == rwl->wr_owner) {
/* Write unlock */ /* Write unlock */
rwl->wr_owner = NULL; rwl->wr_owner = NULL;
sys_sem_give(&rwl->reader_active); (void)sys_sem_give(&rwl->reader_active);
sys_sem_give(&rwl->wr_sem); (void)sys_sem_give(&rwl->wr_sem);
} else { } else {
/* Read unlock */ /* Read unlock */
sys_sem_give(&rwl->rd_sem); (void)sys_sem_give(&rwl->rd_sem);
if (sys_sem_count_get(&rwl->rd_sem) == CONCURRENT_READER_LIMIT) { if (sys_sem_count_get(&rwl->rd_sem) == CONCURRENT_READER_LIMIT) {
/* Last read lock, unlock writer */ /* Last read lock, unlock writer */
sys_sem_give(&rwl->reader_active); (void)sys_sem_give(&rwl->reader_active);
} }
} }
return 0; return 0;
@ -350,9 +350,9 @@ static uint32_t read_lock_acquire(struct posix_rwlock *rwl, int32_t timeout)
uint32_t ret = 0U; uint32_t ret = 0U;
if (sys_sem_take(&rwl->wr_sem, SYS_TIMEOUT_MS(timeout)) == 0) { if (sys_sem_take(&rwl->wr_sem, SYS_TIMEOUT_MS(timeout)) == 0) {
sys_sem_take(&rwl->reader_active, K_NO_WAIT); (void)sys_sem_take(&rwl->reader_active, K_NO_WAIT);
sys_sem_take(&rwl->rd_sem, K_NO_WAIT); (void)sys_sem_take(&rwl->rd_sem, K_NO_WAIT);
sys_sem_give(&rwl->wr_sem); (void)sys_sem_give(&rwl->wr_sem);
} else { } else {
ret = EBUSY; ret = EBUSY;
} }
@ -383,7 +383,7 @@ static uint32_t write_lock_acquire(struct posix_rwlock *rwl, int32_t timeout)
if (sys_sem_take(&rwl->reader_active, k_timeout) == 0) { if (sys_sem_take(&rwl->reader_active, k_timeout) == 0) {
rwl->wr_owner = k_current_get(); rwl->wr_owner = k_current_get();
} else { } else {
sys_sem_give(&rwl->wr_sem); (void)sys_sem_give(&rwl->wr_sem);
ret = EBUSY; ret = EBUSY;
} }