diff options
| author | Rich Felker <dalias@aerifal.cx> | 2011-02-19 09:40:07 -0500 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2011-02-19 09:40:07 -0500 |
| commit | 69ecbd0f3188be97f91cc0d6415836d23e88f7fc (patch) | |
| tree | 0d74e8a311a2ec1a8d0be9fa0f09ac09004a448f /src/temp/mkdtemp.c | |
| parent | 2e6239dd064d201c6e1b0f589bae9ff27949d2eb (diff) | |
| download | musl-69ecbd0f3188be97f91cc0d6415836d23e88f7fc.tar.gz musl-69ecbd0f3188be97f91cc0d6415836d23e88f7fc.tar.bz2 musl-69ecbd0f3188be97f91cc0d6415836d23e88f7fc.tar.xz musl-69ecbd0f3188be97f91cc0d6415836d23e88f7fc.zip | |
make mktemp match the historic behavior, and update functions that use it
the historic mktemp is supposed to blank the template string on
failure, rather than returning 0. just zero the first character so
that mkstemp and mkdtemp can still retry with O(1) space requirement.
Diffstat (limited to 'src/temp/mkdtemp.c')
| -rw-r--r-- | src/temp/mkdtemp.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/temp/mkdtemp.c b/src/temp/mkdtemp.c index f2ecc510..76140c77 100644 --- a/src/temp/mkdtemp.c +++ b/src/temp/mkdtemp.c @@ -12,13 +12,14 @@ char *__mktemp(char *); char *mkdtemp(char *template) { - int retries = 100; + int retries = 100, t0 = *template; while (retries--) { - if (!__mktemp(template)) return 0; + if (!*__mktemp(template)) return 0; if (!mkdir(template, 0700)) return template; if (errno != EEXIST) return 0; /* this is safe because mktemp verified * that we have a valid template string */ + template[0] = t0; strcpy(template+strlen(template)-6, "XXXXXX"); } return 0; |