summaryrefslogtreecommitdiff
path: root/save.c
diff options
context:
space:
mode:
authorAlan Coopersmith <alan.coopersmith@oracle.com>2013-11-23 22:44:06 -0800
committerAlan Coopersmith <alan.coopersmith@oracle.com>2013-11-23 23:10:01 -0800
commitd00b208df92061f0049bfcd993ab1f92dc8aa2cb (patch)
tree6b87fb156e3e52b06434ecd50928842b526a6fe2 /save.c
parentd59fe3c0501ec07b725004de86c1a8adac966092 (diff)
Convert sprintf calls to asprintf or snprintf
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'save.c')
-rw-r--r--save.c17
1 files changed, 6 insertions, 11 deletions
diff --git a/save.c b/save.c
index 5d6b4ce..267f806 100644
--- a/save.c
+++ b/save.c
@@ -361,7 +361,7 @@ unique_filename(char *path, char *prefix, int *pFd)
char tempFile[PATH_MAX];
char *tmp;
- sprintf (tempFile, "%s/%sXXXXXX", path, prefix);
+ snprintf (tempFile, sizeof(tempFile), "%s/%sXXXXXX", path, prefix);
tmp = (char *) mktemp (tempFile);
if (tmp)
{
@@ -373,17 +373,12 @@ unique_filename(char *path, char *prefix, int *pFd)
return (NULL);
# endif /* HAVE_MKTEMP */
#else /* HAVE_MKSTEMP */
- char tempFile[PATH_MAX];
- char *ptr;
+ char *tempFile;
- sprintf (tempFile, "%s/%sXXXXXX", path, prefix);
- ptr = (char *)malloc(strlen(tempFile) + 1);
- if (ptr != NULL)
- {
- strcpy(ptr, tempFile);
- *pFd = mkstemp(ptr);
- }
- return ptr;
+ if (asprintf (&tempFile, "%s/%sXXXXXX", path, prefix) == -1)
+ return NULL;
+ *pFd = mkstemp(tempFile);
+ return tempFile;
#endif /* HAVE_MKSTEMP */
}