bryson/pacman
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add an error message if the sandbox user account does not exist

Browse source
This commit is contained in:
Thijs van Dijk 2025-01-08 11:12:59 +01:00
parent ae603e35d7
commit 628e33c0c1
5 changed files with 13 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/libalpm/be_sync.c
View file

@ -153,7 +153,7 @@ int SYMEXPORT alpm_db_update(alpm_handle_t *handle, alpm_list_t *dbs, int force)
syncpath = get_sync_dir(handle);
ASSERT(syncpath != NULL, return -1);
temporary_syncpath = _alpm_temporary_download_dir_setup(syncpath, handle->sandboxuser);
temporary_syncpath = _alpm_temporary_download_dir_setup(handle, syncpath);
ASSERT(temporary_syncpath != NULL, FREE(syncpath); return -1);
/* make sure we have a sane umask */

2
lib/libalpm/dload.c
View file

@ -1301,7 +1301,7 @@ int SYMEXPORT alpm_fetch_pkgurl(alpm_handle_t *handle, const alpm_list_t *urls,
/* find a valid cache dir to download to */
cachedir = _alpm_filecache_setup(handle);
temporary_cachedir = _alpm_temporary_download_dir_setup(cachedir, handle->sandboxuser);
temporary_cachedir = _alpm_temporary_download_dir_setup(handle, cachedir);
ASSERT(temporary_cachedir != NULL, return -1);
for(i = urls; i; i = i->next) {

2
lib/libalpm/sync.c
View file

@ -780,7 +780,7 @@ static int download_files(alpm_handle_t *handle)
alpm_list_t *payloads = NULL;
cachedir = _alpm_filecache_setup(handle);
temporary_cachedir = _alpm_temporary_download_dir_setup(cachedir, handle->sandboxuser);
temporary_cachedir = _alpm_temporary_download_dir_setup(handle, cachedir);
if(temporary_cachedir == NULL) {
ret = -1;
goto finish;

13
lib/libalpm/util.c
View file

@ -949,17 +949,22 @@ const char *_alpm_filecache_setup(alpm_handle_t *handle)
/** Create a temporary directory under the supplied directory.
* The new directory is writable by the download user, and will be
* removed after the download operation has completed.
* @param handle the context handle
* @param dir existing sync or cache directory
* @param user download user name
* @return pointer to a sub-directory writable by the download user inside the existing directory.
*/
char *_alpm_temporary_download_dir_setup(const char *dir, const char *user)
char *_alpm_temporary_download_dir_setup(alpm_handle_t *handle, const char *dir)
{
struct passwd const *pw = NULL;
ASSERT(dir != NULL, return NULL);
if(user != NULL) {
ASSERT((pw = getpwnam(user)) != NULL, return NULL);
if(handle->sandboxuser != NULL) {
pw = getpwnam(handle->sandboxuser);
if ( pw == NULL ) {
_alpm_log(handle, ALPM_LOG_ERROR,
_("could not create temporary download dir: user '%s' does not exist\n"), handle->sandboxuser);
return NULL;
}
}
const char template[] = "download-XXXXXX";

2
lib/libalpm/util.h
View file

@ -139,7 +139,7 @@ char *_alpm_filecache_find(alpm_handle_t *handle, const char *filename);
/* Checks whether a file exists in cache */
int _alpm_filecache_exists(alpm_handle_t *handle, const char *filename);
const char *_alpm_filecache_setup(alpm_handle_t *handle);
char *_alpm_temporary_download_dir_setup(const char *dir, const char *user);
char *_alpm_temporary_download_dir_setup(alpm_handle_t *handle, const char *dir);
void _alpm_remove_temporary_download_dir(const char *dir);
/* Unlike many uses of alpm_pkgvalidation_t, _alpm_test_checksum expects