OSDN Git Service

lckpwdf.c: add back lost return
authorPeter S. Mazinger <ps.m@gmx.net>
Mon, 21 Mar 2011 16:09:40 +0000 (17:09 +0100)
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Fri, 15 Jun 2012 12:00:30 +0000 (14:00 +0200)
2 earlier (2006 and 2009) commits killed the proper return: -1 for failure, 0 on success

Signed-off-by: Peter S. Mazinger <ps.m@gmx.net>
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
libc/pwd_grp/lckpwdf.c

index adbc616..3c54495 100644 (file)
@@ -51,6 +51,7 @@ lckpwdf (void)
   struct sigaction new_act;            /* New signal action.  */
   struct flock fl;                     /* Information struct for locking.  */
   int result;
+  int rv = -1;
 
   if (lock_fd != -1)
     /* Still locked by own process.  */
@@ -111,11 +112,13 @@ lckpwdf (void)
   if (result < 0) {
     close(lock_fd);
     lock_fd = -1;
+    goto DONE;
   }
+  rv = 0;
 
 DONE:
   __UCLIBC_MUTEX_UNLOCK(mylock);
-  return 0; /* TODO: return result? */
+  return rv;
 }