- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] package.c, fix incorrect buffersize
by Rikard Falkeborn 02 Nov '15

02 Nov '15

Correct title_suffix_len to be the actual number of elements in the string (including the NUL-terminator) instead of the size of a pointer. Note that wmemcpy blindly copies the number of wide characters it is told to copy (no check for NUL-terminating character), so this previously copied data outside of title_suffix. Signed-off-by: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> --- src/pacman/package.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/pacman/package.c b/src/pacman/package.c index dbd23f5..59f4327 100644 --- a/src/pacman/package.c +++ b/src/pacman/package.c @@ -79,6 +79,8 @@ enum { * potential growth. */ #define TITLE_MAXLEN 50 +#define TITLE_SUFFIX L" :" +#define TITLE_SUFFIX_LEN (ARRAYSIZE(TITLE_SUFFIX)) static char titles[_T_MAX][TITLE_MAXLEN * sizeof(wchar_t)]; @@ -90,9 +92,7 @@ static void make_aligned_titles(void) { unsigned int i; size_t max = 0; - static const wchar_t *title_suffix = L" :"; - static const size_t title_suffix_len = sizeof(title_suffix); - wchar_t wbuf[ARRAYSIZE(titles)][TITLE_MAXLEN + title_suffix_len]; + wchar_t wbuf[ARRAYSIZE(titles)][TITLE_MAXLEN + TITLE_SUFFIX_LEN]; size_t wlen[ARRAYSIZE(wbuf)]; char *buf[ARRAYSIZE(wbuf)]; buf[T_ARCHITECTURE] = _("Architecture"); @@ -133,7 +133,7 @@ static void make_aligned_titles(void) for(i = 0; i < ARRAYSIZE(wbuf); i++) { wmemset(wbuf[i] + wlen[i], L' ', max - wlen[i]); - wmemcpy(wbuf[i] + max, title_suffix, title_suffix_len); + wmemcpy(wbuf[i] + max, TITLE_SUFFIX, TITLE_SUFFIX_LEN); wcstombs(titles[i], wbuf[i], sizeof(wbuf[i])); } } -- 2.6.2

4 17

[pacman-dev] [PATCH] add detail to broken dependency errors
by Andrew Gregory 02 Nov '15

02 Nov '15

The difference between a sync target having an unmet dependency and breaking a dependency for an installed package is a common source of confusion. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/remove.c | 3 ++- src/pacman/sync.c | 27 ++++++++++++++++++++++----- 2 files changed, 24 insertions(+), 6 deletions(-) diff --git a/src/pacman/remove.c b/src/pacman/remove.c index 069f236..e41e7f5 100644 --- a/src/pacman/remove.c +++ b/src/pacman/remove.c @@ -122,7 +122,8 @@ int pacman_remove(alpm_list_t *targets) for(i = data; i; i = alpm_list_next(i)) { alpm_depmissing_t *miss = i->data; char *depstring = alpm_dep_compute_string(miss->depend); - colon_printf(_("%s: requires %s\n"), miss->target, depstring); + colon_printf(_("%s: removing %s breaks dependency '%s'\n"), + miss->target, miss->causingpkg, depstring); free(depstring); alpm_depmissing_free(miss); } diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 2a0f4b5..b13f76a 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -707,6 +707,26 @@ static int sync_trans(alpm_list_t *targets) return sync_prepare_execute(); } +static void print_broken_dep(alpm_depmissing_t *miss) +{ + char *depstring = alpm_dep_compute_string(miss->depend); + alpm_list_t *trans_add = alpm_trans_get_add(config->handle); + alpm_pkg_t *pkg; + if(miss->causingpkg == NULL) { + /* package being installed/upgraded has unresolved dependency */ + colon_printf(_("%s: requires %s\n"), miss->target, depstring); + } else if((pkg = alpm_pkg_find(trans_add, miss->causingpkg))) { + /* upgrading a package breaks a local dependency */ + colon_printf(_("%s: installing %s (%s) breaks dependency '%s'\n"), + miss->target, miss->causingpkg, alpm_pkg_get_version(pkg), depstring); + } else { + /* removing a package breaks a local dependency */ + colon_printf(_("%s: removing %s breaks dependency '%s'\n"), + miss->target, miss->causingpkg, depstring); + } + free(depstring); +} + int sync_prepare_execute(void) { alpm_list_t *i, *packages, *data = NULL; @@ -727,11 +747,8 @@ int sync_prepare_execute(void) break; case ALPM_ERR_UNSATISFIED_DEPS: for(i = data; i; i = alpm_list_next(i)) { - alpm_depmissing_t *miss = i->data; - char *depstring = alpm_dep_compute_string(miss->depend); - colon_printf(_("%s: requires %s\n"), miss->target, depstring); - free(depstring); - alpm_depmissing_free(miss); + print_broken_dep(i->data); + alpm_depmissing_free(i->data); } break; case ALPM_ERR_CONFLICTING_DEPS: -- 2.6.2

1 0

[pacman-dev] [PATCH] add --quiet to -Fh
by Andrew Gregory 02 Nov '15

02 Nov '15

Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/pacman.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 019b0c2..e3d4621 100644 --- a/src/pacman/pacman.c +++ b/src/pacman/pacman.c @@ -177,6 +177,7 @@ static void usage(int op, const char * const myname) } else if(op == PM_OP_FILES) { addlist(_(" -l, --list list the files owned by the queried package\n")); addlist(_(" -o, --owns <file> query the package that owns <file>\n")); + addlist(_(" -q, --quiet show less information for query and search\n")); addlist(_(" -s, --search <file> search package file names for matching strings\n")); addlist(_(" -x, --regex enable searching using regular expressions\n")); addlist(_(" -y, --refresh download fresh package databases from the server\n" -- 2.6.2

1 0

[pacman-dev] [PATCH] files_search: reset found for each target
by Andrew Gregory 02 Nov '15

02 Nov '15

Otherwise any invalid targets following a successful match will not get an error message. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/files.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index c2231cd..3dc915a 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -178,10 +178,11 @@ static void dump_file_list(alpm_pkg_t *pkg) { static int files_list(alpm_list_t *syncs, alpm_list_t *targets) { alpm_list_t *i, *j; - int ret = 0, found = 0; + int ret = 0; if(targets != NULL) { for(i = targets; i; i = alpm_list_next(i)) { + int found = 0; char *targ = i->data; char *repo = NULL; char *c = strchr(targ, '/'); -- 2.6.2

1 0

[pacman-dev] [PATCH] files_search: free compiled regex
by Andrew Gregory 02 Nov '15

02 Nov '15

Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/files.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/pacman/files.c b/src/pacman/files.c index ef48f96..c2231cd 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -141,6 +141,9 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { } } + if(regex) { + regfree(&reg); + } notfound: if(!found) { -- 2.6.2

1 0

[pacman-dev] [PATCH] files: do not unnecessarily strdup targets
by Andrew Gregory 02 Nov '15

02 Nov '15

Targets are never modified so we can just use the original copy. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/files.c | 33 +++++++++------------------------ 1 file changed, 9 insertions(+), 24 deletions(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index 9536005..ef48f96 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -33,19 +33,13 @@ static int files_fileowner(alpm_list_t *syncs, alpm_list_t *targets) { alpm_list_t *t; for(t = targets; t; t = alpm_list_next(t)) { - char *filename = NULL, *f; + char *filename = t->data; int found = 0; alpm_list_t *s; - size_t len; + size_t len = strlen(filename); - if((filename = strdup(t->data)) == NULL) { - goto notfound; - } - - len = strlen(filename); - f = filename; - while(len > 1 && f[0] == '/') { - f = f + 1; + while(len > 1 && filename[0] == '/') { + filename++; len--; } @@ -58,10 +52,10 @@ static int files_fileowner(alpm_list_t *syncs, alpm_list_t *targets) { alpm_pkg_t *pkg = p->data; alpm_filelist_t *files = alpm_pkg_get_files(pkg); - if(alpm_filelist_contains(files, f)) { + if(alpm_filelist_contains(files, filename)) { if(!config->quiet) { - printf(_("%s is owned by %s/%s %s\n"), f, + printf(_("%s is owned by %s/%s %s\n"), filename, alpm_db_get_name(repo), alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg)); } else { @@ -73,9 +67,6 @@ static int files_fileowner(alpm_list_t *syncs, alpm_list_t *targets) { } } - free(filename); - -notfound: if(!found) { ret++; } @@ -90,19 +81,14 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { const colstr_t *colstr = &config->colstr; for(t = targets; t; t = alpm_list_next(t)) { - char *targ = NULL; + char *targ = t->data; alpm_list_t *s; int found = 0; regex_t reg; - if((targ = strdup(t->data)) == NULL) { - goto notfound; - } - if(regex) { if(regcomp(&reg, targ, REG_EXTENDED | REG_NOSUB | REG_ICASE | REG_NEWLINE) != 0) { /* TODO: error message */ - free(targ); goto notfound; } } @@ -129,7 +115,7 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { m = strcmp(c + 1, targ); } if(m == 0) { - match = alpm_list_add(match, strdup(files->files[f].name)); + match = alpm_list_add(match, files->files[f].name); found = 1; } } @@ -149,13 +135,12 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { c = ml->data; printf(" %s\n", c); } - FREELIST(match); } + alpm_list_free(match); } } } - free(targ); notfound: if(!found) { -- 2.6.2

1 0

[pacman-dev] [PATCH] files.c: add vim modeline
by Andrew Gregory 02 Nov '15

02 Nov '15

Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- src/pacman/files.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index 31ce9e0..9536005 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -289,6 +289,7 @@ int pacman_files(alpm_list_t *targets) return files_list(files_dbs, targets); } - return 0; } + +/* vim: set noet: */ -- 2.6.2

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.2.1-242-gd8621b9
by allan＠archlinux.org 28 Oct '15

28 Oct '15

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, master has been updated via d8621b981e100ecf46d260e1a85cc13d202cafda (commit) via dcc1b22cb33a7a9d9477ffc84f833317860bf751 (commit) via 1142a32c7f6f8ab82528581277b7c7cac6b2d1ae (commit) via 960b64553db7c738202d5108abff77af0bdb87b5 (commit) via 6fdc589fc6d5598c2ce3f3e029594417dcb25b39 (commit) via 5b6526a2adad6140c3878838ae0e7b61d5ee7594 (commit) via b46bdeea146b19f026b2bb2ad4d281aa89fff777 (commit) via 3c671270180163798cb0d08ea163e7100ab21ee6 (commit) via 0c5dbdbfeca8a26d015230a8edd6d7c4c91d386a (commit) via 4c5bf09eec4835f8dfd300215f16289d106266d5 (commit) via 456ebe8f8eee82c467805613f107edd84e73ac26 (commit) via 9809102237f89b73a120250260bf3f68d6bc4ba9 (commit) via e28aff4d87f9456c213359adaf7299a882cdcb04 (commit) via aa8a674b6b2c8474a1111b16806c27d4562b5d80 (commit) via 19d373c9b95ff5c9ee6d5a4cdb097670f69068d4 (commit) via 4b3df10d5d53af94475e006c71ed262efcab6f0f (commit) via 2c72c8b822d21c45a8020e49f03691830f3bcff2 (commit) via 2627b423ff786edd60123a2670a69841fe4cb5a4 (commit) via 70e6875ad9afd6ee0e26c7fbe283466f7a6767e5 (commit) via 5d4a3f101ca15bb1ae7944ec305c7f10c9f504d0 (commit) via 5301d3fe8fc3fc0a65f6361ba3b4afcdcb9c2cf8 (commit) via 29d4dcf767b7325ac7e9bbaa4c0b45e3e3ee2c73 (commit) via 501242a35bce01f19312b774c6364b4edaaec619 (commit) from 0ee1beca12f6070d406bbf09c3036124018e57bf (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit d8621b981e100ecf46d260e1a85cc13d202cafda Author: Allan McRae <allan(a)archlinux.org> Date: Wed Oct 28 17:04:08 2015 +1000 scripts/po: fix translated file name This file was renamed during review and its entry in POTFILES.in was not updated. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit dcc1b22cb33a7a9d9477ffc84f833317860bf751 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Wed Oct 28 00:04:05 2015 -0400 _alpm_hook_run: check path length before copying If a hook path equals or exceeds PATH_MAX characters the path will be left unterminated. Pre-calculating the path length also allows us to use the more efficient memcpy over strncpy. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 1142a32c7f6f8ab82528581277b7c7cac6b2d1ae Author: Benjamin Yates <benjamin(a)rqdq.com> Date: Tue Oct 27 16:22:57 2015 -0400 makepkg: fix $pkgname element duplication run_split_packaging did not preserve the $pkgname array correctly, and would create duplicate entries in the list during restore. After restoring the backup (a b c) would become (a b c b c). This probably went unnoticed because during --install, pacman would reconcile the duplicates. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 960b64553db7c738202d5108abff77af0bdb87b5 Author: Allan McRae <allan(a)archlinux.org> Date: Tue Oct 27 20:44:57 2015 +1000 libmakepkg: fix pkgver checking Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 6fdc589fc6d5598c2ce3f3e029594417dcb25b39 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Tue Oct 27 00:47:32 2015 -0400 add alpm-hooks man page Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 5b6526a2adad6140c3878838ae0e7b61d5ee7594 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Tue Oct 27 00:47:31 2015 -0400 hook.c: print invalid option name Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit b46bdeea146b19f026b2bb2ad4d281aa89fff777 Author: Andrew Gregory <andrew.gregory.8(a)gmail.com> Date: Tue Oct 27 00:47:30 2015 -0400 order hooks by file name Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 3c671270180163798cb0d08ea163e7100ab21ee6 Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Tue Oct 27 09:38:10 2015 +0100 Align titles automatically in information display Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 0c5dbdbfeca8a26d015230a8edd6d7c4c91d386a Author: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Date: Sun Oct 25 12:39:27 2015 +0100 Alpm, check for NULL in free-functions Also, use FREE() instead of free() in _alpm_backup_free() to set the pointers to NULL. Signed-off-by: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 4c5bf09eec4835f8dfd300215f16289d106266d5 Author: Mohammad Alsaleh <ce.mohammad.alsaleh(a)gmail.com> Date: Fri Oct 23 01:38:49 2015 +0300 zsh_completion: Add --asdeps/--asexplicit to -U options Signed-off-by: Mohammad Alsaleh <CE.Mohammad.AlSaleh(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 456ebe8f8eee82c467805613f107edd84e73ac26 Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Wed Oct 21 11:11:17 2015 +0200 Use ARRAYSIZE macro for non-string array size computation Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 9809102237f89b73a120250260bf3f68d6bc4ba9 Author: Mohammad Alsaleh <ce.mohammad.alsaleh(a)gmail.com> Date: Thu Oct 22 02:27:09 2015 +0300 makepkg: Fix hard-coded debug suffix Signed-off-by: Mohammad Alsaleh <CE.Mohammad.AlSaleh(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit e28aff4d87f9456c213359adaf7299a882cdcb04 Author: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Date: Mon Aug 10 21:42:39 2015 +0200 pacsort: clean up if error * free memory * close open file Signed-off-by: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit aa8a674b6b2c8474a1111b16806c27d4562b5d80 Author: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Date: Mon Aug 10 21:42:38 2015 +0200 pacsort: don't overwrite memory if realloc fails That makes it impossible to free it later. Signed-off-by: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 19d373c9b95ff5c9ee6d5a4cdb097670f69068d4 Author: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Date: Mon Aug 10 21:42:37 2015 +0200 pacsort: handle failing list_add Since it can fail, check the return value. If it fails, we need to free the memory of the object we wanted to add to the list. Signed-off-by: Rikard Falkeborn <rikard.falkeborn(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 4b3df10d5d53af94475e006c71ed262efcab6f0f Author: Pierre Neidhardt <ambrevar(a)gmail.com> Date: Tue Oct 20 15:30:32 2015 +0200 HACKING: Allow the use of 'sizeof' on values Signed-off-by: Pierre Neidhardt <ambrevar(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 2c72c8b822d21c45a8020e49f03691830f3bcff2 Author: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> Date: Tue Sep 15 22:17:04 2015 +0200 makepkg: Set CCACHE_BASEDIR to make paths in $srcdir relative Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 2627b423ff786edd60123a2670a69841fe4cb5a4 Author: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> Date: Tue Sep 15 22:17:03 2015 +0200 makepkg: Correctly layer distcc and ccache ccache expects further compiler wrappers to be specified via CCACHE_PREFIX. Otherwise, ccache will hash the wrapper executable instead of the real one. Signed-off-by: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 70e6875ad9afd6ee0e26c7fbe283466f7a6767e5 Author: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> Date: Mon Oct 19 03:33:24 2015 +0200 libmakepkg: Add check_buildoption for distcc and ccache makepkg used to check OPTIONS too, which could override BUILDENV. Implement a new function that handles these options more like OPTIONS. This also reduces code duplication a bit. Signed-off-by: Jan Alexander Steffens (heftig) <jan.steffens(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 5d4a3f101ca15bb1ae7944ec305c7f10c9f504d0 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Oct 19 14:25:35 2015 +1000 makepkg: separate "sourcefile" into two words Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 5301d3fe8fc3fc0a65f6361ba3b4afcdcb9c2cf8 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Oct 19 13:47:19 2015 +1000 Remove space before ellipses Makes all use of ellipses consistent... Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 29d4dcf767b7325ac7e9bbaa4c0b45e3e3ee2c73 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Oct 19 13:43:26 2015 +1000 Pluralize malloc failure string Not particularly useful in English (will always be plural), but useful in languages that have multiple plural forms. Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 501242a35bce01f19312b774c6364b4edaaec619 Author: Allan McRae <allan(a)archlinux.org> Date: Sun Oct 18 16:05:42 2015 +1000 Ignore cov-int directory for Coverity scan Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: .gitignore | 1 + HACKING | 14 +-- contrib/zsh_completion.in | 2 + doc/.gitignore | 1 + doc/Makefile.am | 4 + doc/alpm-hooks.5.txt | 118 ++++++++++++++++++ lib/libalpm/backup.c | 7 +- lib/libalpm/conflict.c | 2 + lib/libalpm/db.c | 1 + lib/libalpm/delta.c | 1 + lib/libalpm/deps.c | 2 + lib/libalpm/graph.c | 1 + lib/libalpm/hook.c | 37 ++++-- lib/libalpm/pkghash.c | 2 +- scripts/libmakepkg/lint_pkgbuild/pkgver.sh.in | 4 +- scripts/libmakepkg/source/bzr.sh.in | 4 +- scripts/libmakepkg/util/option.sh | 34 +++++ scripts/makepkg.sh.in | 36 +++--- scripts/pacman-key.sh.in | 2 +- scripts/po/POTFILES.in | 2 +- scripts/repo-add.sh.in | 2 +- src/common/util-common.h | 2 + src/pacman/conf.c | 4 +- src/pacman/package.c | 173 +++++++++++++++++++++----- src/pacman/pacman.c | 2 +- src/pacman/sync.c | 2 +- src/pacman/util.c | 2 +- src/util/pacsort.c | 27 ++-- 28 files changed, 400 insertions(+), 89 deletions(-) create mode 100644 doc/alpm-hooks.5.txt hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] _alpm_hook_run: check path length before copying
by Andrew Gregory 28 Oct '15

28 Oct '15

If a hook path equals or exceeds PATH_MAX characters the path will be left unterminated. Pre-calculating the path length also allows us to use the more efficient memcpy over strncpy. Signed-off-by: Andrew Gregory <andrew.gregory.8(a)gmail.com> --- lib/libalpm/hook.c | 27 +++++++++++++++++++-------- 1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/lib/libalpm/hook.c b/lib/libalpm/hook.c index 668e086..b440e08 100644 --- a/lib/libalpm/hook.c +++ b/lib/libalpm/hook.c @@ -621,30 +621,41 @@ int _alpm_hook_run(alpm_handle_t *handle, enum _alpm_hook_when_t when) struct dirent entry, *result; DIR *d; - if(!(d = opendir(i->data))) { + if((dirlen = strlen(i->data)) >= PATH_MAX) { + _alpm_log(handle, ALPM_LOG_ERROR, _("could not open directory: %s: %s\n"), + (char *)i->data, strerror(ENAMETOOLONG)); + ret = -1; + continue; + } + memcpy(path, i->data, dirlen + 1); + + if(!(d = opendir(path))) { if(errno == ENOENT) { continue; } else { - _alpm_log(handle, ALPM_LOG_ERROR, _("could not open directory: %s: %s\n"), - (char *)i->data, strerror(errno)); + _alpm_log(handle, ALPM_LOG_ERROR, + _("could not open directory: %s: %s\n"), path, strerror(errno)); ret = -1; continue; } } - strncpy(path, i->data, PATH_MAX); - dirlen = strlen(i->data); - while((err = readdir_r(d, &entry, &result)) == 0 && result) { struct _alpm_hook_cb_ctx ctx = { handle, NULL }; struct stat buf; - size_t name_len = strlen(entry.d_name); + size_t name_len; if(strcmp(entry.d_name, ".") == 0 || strcmp(entry.d_name, "..") == 0) { continue; } - strncpy(path + dirlen, entry.d_name, PATH_MAX - dirlen); + if((name_len = strlen(entry.d_name)) >= PATH_MAX - dirlen) { + _alpm_log(handle, ALPM_LOG_ERROR, _("could not open file: %s%s: %s\n"), + path, entry.d_name, strerror(ENAMETOOLONG)); + ret = -1; + continue; + } + memcpy(path + dirlen, entry.d_name, name_len + 1); if(name_len < suflen || strcmp(entry.d_name + name_len - suflen, suffix) != 0) { -- 2.6.2

1 0

[pacman-dev] [PATCH] makepkg: fix $pkgname element duplication
by Benjamin Yates 28 Oct '15

28 Oct '15

run_split_packaging did not preserve the $pkgname array correctly, and would create duplicate entries in the list during restore. After restoring the backup (a b c) would become (a b c b c). This probably went unnoticed because during --install, pacman would reconcile the duplicates. --- scripts/makepkg.sh.in | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 6ededa3..224f089 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1664,7 +1664,7 @@ restore_package_variables() { } run_split_packaging() { - local pkgname_backup=${pkgname[@]} + local pkgname_backup=("${pkgname[@]}") for pkgname in ${pkgname_backup[@]}; do pkgdir="$pkgdirbase/$pkgname" mkdir "$pkgdir" @@ -1676,7 +1676,7 @@ run_split_packaging() { create_debug_package restore_package_variables done - pkgname=${pkgname_backup[@]} + pkgname=("${pkgname_backup[@]}") } m4_include(library/parseopts.sh) -- 1.7.1

2 1