[pacman-dev] [PATCH] Move creating a temporary directory into a utility function

[Allan McRae]

---

This will be used for downloading sync directories into a temporary location
so they can be validated before replacing exisiting sync dbs.

 lib/libalpm/trans.c | 11 +----------
 lib/libalpm/util.c  | 28 ++++++++++++++++++++++++++++
 lib/libalpm/util.h  |  1 +
 3 files changed, 30 insertions(+), 10 deletions(-)

diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c
index 14f7d5e3..e4c8e404 100644
--- a/lib/libalpm/trans.c
+++ b/lib/libalpm/trans.c
@@ -353,16 +353,7 @@ int _alpm_runscriptlet(alpm_handle_t *handle, const char 
*filepath,
        strcpy(arg1, "-c");
 
        /* create a directory in $root/tmp/ for copying/extracting the 
scriptlet */
-       len = strlen(handle->root) + strlen("tmp/alpm_XXXXXX") + 1;
-       MALLOC(tmpdir, len, RET_ERR(handle, ALPM_ERR_MEMORY, -1));
-       snprintf(tmpdir, len, "%stmp/", handle->root);
-       if(access(tmpdir, F_OK) != 0) {
-               _alpm_makepath_mode(tmpdir, 01777);
-       }
-       snprintf(tmpdir, len, "%stmp/alpm_XXXXXX", handle->root);
-       if(mkdtemp(tmpdir) == NULL) {
-               _alpm_log(handle, ALPM_LOG_ERROR, _("could not create temp 
directory\n"));
-               free(tmpdir);
+       if((len = _alpm_mkdtemp(handle, &tmpdir)) == 0) {
                return 1;
        }
 
diff --git a/lib/libalpm/util.c b/lib/libalpm/util.c
index d12a4403..0ec9f650 100644
--- a/lib/libalpm/util.c
+++ b/lib/libalpm/util.c
@@ -142,6 +142,34 @@ done:
        return ret;
 }
 
+
+/** Creates a temporary directory of form /$root/tmp/alpm_XXXXXX.
+ * @param handle the context handle
+ * @param tmpdir pointer to storage of created directory path
+ * @return path length on success, 0 on error
+ */
+int _alpm_mkdtemp(alpm_handle_t *handle, char **tmpdir)
+{
+       size_t len;
+
+       ASSERT(tmpdir, RET_ERR(handle, ALPM_ERR_WRONG_ARGS, 0));
+
+       len = strlen(handle->root) + strlen("tmp/alpm_XXXXXX") + 1;
+       MALLOC(*tmpdir, len, RET_ERR(handle, ALPM_ERR_MEMORY, 0));
+       snprintf(*tmpdir, len, "%stmp/", handle->root);
+       if(access(*tmpdir, F_OK) != 0) {
+               _alpm_makepath_mode(*tmpdir, 01777);
+       }
+       snprintf(*tmpdir, len, "%stmp/alpm_XXXXXX", handle->root);
+       if(mkdtemp(*tmpdir) == NULL) {
+               _alpm_log(handle, ALPM_LOG_ERROR, _("could not create temp 
directory\n"));
+               free(tmpdir);
+               return 0;
+       }
+
+       return len;
+}
+
 /** Copies a file.
  * @param src file path to copy from
  * @param dest file path to copy to
diff --git a/lib/libalpm/util.h b/lib/libalpm/util.h
index 71dadc2c..12c7fbc7 100644
--- a/lib/libalpm/util.h
+++ b/lib/libalpm/util.h
@@ -111,6 +111,7 @@ struct archive_read_buffer {
 
 int _alpm_makepath(const char *path);
 int _alpm_makepath_mode(const char *path, mode_t mode);
+int _alpm_mkdtemp(alpm_handle_t *handle, char **tmpdir);
 int _alpm_copyfile(const char *src, const char *dest);
 size_t _alpm_strip_newline(char *str, size_t len);
 
-- 
2.24.1