diff options
| author | liuxianliang <[email protected]> | 2022-07-05 10:36:53 +0800 |
|---|---|---|
| committer | guo <[email protected]> | 2022-07-07 16:05:49 +0800 |
| commit | a305edd4db9b1c7d1706fb717bcd9923775afed4 (patch) | |
| tree | 8e6542c9f63fb9e9cf2d7bfaad1ad4f5f6f9bca5 | |
| parent | d3139706e45d0dfc966d7fd8db1d9c987353e35d (diff) | |
[fix] memory leak when the value of posix semaphore is zero.
| -rw-r--r-- | components/libc/posix/ipc/semaphore.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/components/libc/posix/ipc/semaphore.c b/components/libc/posix/ipc/semaphore.c index d5b1a70279..6a4ae51e57 100644 --- a/components/libc/posix/ipc/semaphore.c +++ b/components/libc/posix/ipc/semaphore.c @@ -39,7 +39,9 @@ static void posix_sem_delete(sem_t *psem) posix_sem_list = psem->next; rt_sem_delete(psem->sem); - rt_free(psem); + + if(psem->unamed == 0) + rt_free(psem); return; } @@ -55,7 +57,9 @@ static void posix_sem_delete(sem_t *psem) /* delete RT-Thread mqueue */ rt_sem_delete(psem->sem); - rt_free(psem); + + if(psem->unamed == 0) + rt_free(psem); return ; } @@ -107,8 +111,6 @@ RTM_EXPORT(sem_close); int sem_destroy(sem_t *sem) { - rt_err_t result; - if ((!sem) || !(sem->unamed)) { rt_set_errno(EINVAL); @@ -118,8 +120,7 @@ int sem_destroy(sem_t *sem) /* lock posix semaphore list */ rt_sem_take(&posix_sem_lock, RT_WAITING_FOREVER); - result = rt_sem_trytake(sem->sem); - if (result != RT_EOK) + if(rt_list_len(&sem->sem->parent.suspend_thread) != 0) { rt_sem_release(&posix_sem_lock); rt_set_errno(EBUSY); |