Fix: compat_futex_noasync: don't override return value
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 14 Sep 2015 00:47:10 +0000 (20:47 -0400)
committerMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Mon, 14 Sep 2015 00:47:10 +0000 (20:47 -0400)
Fix error reported by Coverity:
** CID 1324336:  Code maintainability issues  (UNUSED_VALUE)
/compat_futex.c: 99 in compat_futex_noasync()

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
compat_futex.c

index a3571343ca37d086e77620da1ed0dd5e9fd71778..0cc2956b66bcf35a5fe0786844e9d133743e6690 100644 (file)
@@ -54,7 +54,7 @@ pthread_cond_t __urcu_compat_futex_cond = PTHREAD_COND_INITIALIZER;
 int compat_futex_noasync(int32_t *uaddr, int op, int32_t val,
        const struct timespec *timeout, int32_t *uaddr2, int32_t val3)
 {
-       int ret;
+       int ret, lockret;
 
        /*
         * Check if NULL. Don't let users expect that they are taken into
@@ -69,9 +69,9 @@ int compat_futex_noasync(int32_t *uaddr, int op, int32_t val,
         */
        cmm_smp_mb();
 
-       ret = pthread_mutex_lock(&__urcu_compat_futex_lock);
-       if (ret) {
-               errno = ret;
+       lockret = pthread_mutex_lock(&__urcu_compat_futex_lock);
+       if (lockret) {
+               errno = lockret;
                ret = -1;
                goto end;
        }
@@ -98,9 +98,9 @@ int compat_futex_noasync(int32_t *uaddr, int op, int32_t val,
                errno = EINVAL;
                ret = -1;
        }
-       ret = pthread_mutex_unlock(&__urcu_compat_futex_lock);
-       if (ret) {
-               errno = ret;
+       lockret = pthread_mutex_unlock(&__urcu_compat_futex_lock);
+       if (lockret) {
+               errno = lockret;
                ret = -1;
        }
 end:
This page took 0.034607 seconds and 4 git commands to generate.