[pacman-dev] [PATCH] Use sane umask for repo db downloads

[Allan McRae]

Fixes FS#23343.

Signed-off-by: Allan McRae <al...@archlinux.org>
---
 lib/libalpm/be_sync.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c
index 6c0d2d6..216213b 100644
--- a/lib/libalpm/be_sync.c
+++ b/lib/libalpm/be_sync.c
@@ -84,6 +84,7 @@ int SYMEXPORT alpm_db_update(int force, pmdb_t *db)
        struct stat buf;
        size_t len;
        int ret;
+       mode_t oldmask;
 
        ALPM_LOG_FUNC;
 
@@ -104,6 +105,9 @@ int SYMEXPORT alpm_db_update(int force, pmdb_t *db)
        MALLOC(syncpath, len, RET_ERR(PM_ERR_MEMORY, -1));
        sprintf(syncpath, "%s%s", dbpath, "sync/");
 
+       /* make sure we have a sane umask */
+       oldmask = umask(0022);
+
        if(stat(syncpath, &buf) != 0) {
                _alpm_log(PM_LOG_DEBUG, "database dir '%s' does not exist, 
creating it\n",
                                syncpath);
@@ -124,6 +128,7 @@ int SYMEXPORT alpm_db_update(int force, pmdb_t *db)
        ret = _alpm_download_single_file(dbfile, db->servers, syncpath, force);
        free(dbfile);
        free(syncpath);
+       umask(oldmask);
 
        if(ret == 1) {
                /* files match, do nothing */
-- 
1.7.4.1