From: Rich Felker <dalias@aerifal.cx>
Date: Fri, 18 Feb 2011 21:32:33 +0000 (-0500)
Subject: reformat mkstemp like mkdtemp
X-Git-Tag: v0.7.0~100
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=3e9e30166f22f8fb0d5664500bb52a00d1a3c6a3;p=oweals%2Fmusl.git

reformat mkstemp like mkdtemp

this is cleaner and makes it easy to impose a limit on the number of
retries later if it seems desirable to do so.
---

diff --git a/src/temp/mkstemp.c b/src/temp/mkstemp.c
index 5e8bb934..34642569 100644
--- a/src/temp/mkstemp.c
+++ b/src/temp/mkstemp.c
@@ -12,17 +12,15 @@ char *__mktemp(char *);
 int mkstemp(char *template)
 {
 	int fd;
-retry:
-	if (!__mktemp(template)) return -1;
-	fd = open(template, O_RDWR | O_CREAT | O_EXCL, 0600);
-	if (fd >= 0) return fd;
-	if (errno == EEXIST) {
+	for (;;) {
+		if (!__mktemp(template)) return 0;
+		if ((fd = open(template, O_RDWR | O_CREAT | O_EXCL, 0600))>=0)
+			return fd;
+		if (errno != EEXIST) return -1;
 		/* this is safe because mktemp verified
 		 * that we have a valid template string */
 		strcpy(template+strlen(template)-6, "XXXXXX");
-		goto retry;
 	}
-	return -1;
 }
 
 LFS64(mkstemp);