Browse Source

fix temp file leak in sem_open on successful creation of new semaphore

Rich Felker 11 years ago
parent
commit
7c20a11801
1 changed files with 2 additions and 2 deletions
  1. 2 2
      src/thread/sem_open.c

+ 2 - 2
src/thread/sem_open.c

@@ -123,9 +123,9 @@ sem_t *sem_open(const char *name, int flags, ...)
 			goto fail;
 		}
 		close(fd);
-		if (link(tmp, name) == 0) break;
-		e = errno;
+		e = link(tmp, name) ? errno : 0;
 		unlink(tmp);
+		if (!e) break;
 		/* Failure is only fatal when doing an exclusive open;
 		 * otherwise, next iteration will try to open the
 		 * existing file. */