From 1f80c2afb0d826567a9a5a1c3ce76c28883e0e96 Mon Sep 17 00:00:00 2001 From: Filippo Negroni Date: Thu, 25 Feb 2010 10:01:31 +0000 Subject: [PATCH] Fix gitmkdtemp: correct test for mktemp() return value In gitmkdtemp, the return value of mktemp is not tested correctly. mktemp() always returns its 'template' argument, even upon failure. An error is signalled by making the template an empty string. Signed-off-by: Filippo Negroni Signed-off-by: Junio C Hamano --- compat/mkdtemp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/compat/mkdtemp.c b/compat/mkdtemp.c index 34d4b49818..1136119592 100644 --- a/compat/mkdtemp.c +++ b/compat/mkdtemp.c @@ -2,7 +2,7 @@ char *gitmkdtemp(char *template) { - if (!mktemp(template) || mkdir(template, 0700)) + if (!*mktemp(template) || mkdir(template, 0700)) return NULL; return template; }