discover: Use mkstemp rather than tempnam

We're getting warnings from some compilers about tempnam being unsafe.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

diff --git a/discover/paths.c b/discover/paths.c
index aa977fdba1f3d0bcbdbf84356a854877c202a17d..64b218e820d2f06ec508584a7b4a2fddea0fc5b6 100644 (file)
--- a/discover/paths.c
+++ b/discover/paths.c
@@ -46,15 +46,17 @@ char *join_paths(void *alloc_ctx, const char *a, const char *b)
 
 static char *local_name(void *ctx)
 {
 
 static char *local_name(void *ctx)
 {

-       char *tmp, *ret;
+       char *ret, tmp[] = "/tmp/pb-XXXXXX";
+       int fd;

 
 

-       tmp = tempnam(NULL, "pb-");
+       fd = mkstemp(tmp);

 
 

-       if (!tmp)
+       if (fd < 0)

                return NULL;
 
                return NULL;
 

+       close(fd);
+

        ret = talloc_strdup(ctx, tmp);
        ret = talloc_strdup(ctx, tmp);

-       free(tmp);

 
        return ret;
 }
 
        return ret;
 }