- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] sync: fix garbled output in conflict prompt
by Dave Reisner 19 Aug '11

19 Aug '11

$ pacman -S cronie resolving dependencies... looking for inter-conflicts... :: cronie and fcron are in conflict (@.). Remove fcron? [y/N] n $ sudo pacman -S pacman resolving dependencies... looking for inter-conflicts... :: pacman and pacman-git are in conflict (pKÈ). Remove pacman-git? [y/N] Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- lib/libalpm/sync.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 11ee817..6c30c94 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -527,7 +527,7 @@ int _alpm_sync_prepare(alpm_handle_t *handle, alpm_list_t **data) alpm_pkg_t *local = _alpm_db_get_pkgfromcache(handle->db_local, conflict->package2); int doremove = 0; QUESTION(trans, ALPM_TRANS_CONV_CONFLICT_PKG, conflict->package1, - conflict->package2, conflict->reason, &doremove); + conflict->package2, conflict->reason->name, &doremove); if(doremove) { /* append to the removes list */ _alpm_log(handle, ALPM_LOG_DEBUG, "electing '%s' for removal\n", conflict->package2); -- 1.7.6

1 0

[pacman-dev] [PATCH 1/2] lib/dload: prevent possible NULL dereference
by Dave Reisner 18 Aug '11

18 Aug '11

Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- Our (my) download code isn't very clear, but there's an oddball chance this might happen. Pointed out by Lucas. lib/libalpm/dload.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 5464740..3798937 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -396,7 +396,7 @@ cleanup: } } - if((ret == -1 || dload_interrupted) && should_unlink) { + if((ret == -1 || dload_interrupted) && should_unlink && tempfile) { unlink(tempfile); } -- 1.7.6

3 4

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.0.0rc1-57-gc28052e
by dan＠archlinux.org 18 Aug '11

18 Aug '11

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 c28052e45b94437d8c476c648ad6c9f9b8c04f36 (commit) via 2a466c2abc94ce835efac94510659af006bd4330 (commit) via 6cfc4757b98e813428d261dbc185e20618ca83a6 (commit) via 3752edbff48fc472c5944f05f665adb000a521d9 (commit) via d008a816f1125ca44f0b2165b38220047860ecb0 (commit) via 78b63ce7c30f4a20707bce61eeaef913b513f557 (commit) via 231d6cc1e4083e5e49063b22fe6e5b0318b85328 (commit) via 5d291d050ebd714851c6bd85efd49a03d88414f0 (commit) via c4bd476ad13e142fe8323fe74d84b3950b53da17 (commit) via c885a953eb888004f0302ed3eceafef93e2f072f (commit) via 4a7f3bbc469d1f6a8da1c7f310ab518ad841c2b9 (commit) via d18e600952015378c9ba79661597f1c01046f436 (commit) via 0745288c1441c73a0c6ce4279823d8363bcdc62f (commit) from 9f139550f815e2970b8929094e50acfd8b4cc2b7 (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 c28052e45b94437d8c476c648ad6c9f9b8c04f36 Author: Allan McRae <allan(a)archlinux.org> Date: Thu Aug 18 23:01:29 2011 +1000 makepkg: deal with variable substitution when checking sanity If any of pkgver, pkgrel or epoch contained a variable substitution, then it needed to be evaluated before checking its value conformed to the rules. [Dan: add quotes around RHS] Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 2a466c2abc94ce835efac94510659af006bd4330 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 18 00:40:28 2011 -0500 doc/PKGBUILD: update regarding versioned package fields Add the info that versioned replaces are now supported, as well as beefing up some of the other places touching on versioned fields. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 6cfc4757b98e813428d261dbc185e20618ca83a6 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 18 00:24:41 2011 -0500 Convert resolvedep() to use _alpm_depcmp_literal() The whole first loop is trying to check literals only, so teach it to do so. Also, reorder operations to make more sense by putting the strcmp() first in the literal loop, and using a very cheap name_hash check first in the second loop. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3752edbff48fc472c5944f05f665adb000a521d9 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 18 00:23:06 2011 -0500 Fix replacement of provider issue When we switched to using alpm_depcmp() in resolving replacments, we had some interesting behavior with regard to providers and packages not found in repositories. Teach the replacement resolving code to not look at provisions at all to be slightly more sane. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d008a816f1125ca44f0b2165b38220047860ecb0 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 18 00:11:19 2011 -0500 Add an _alpm_depcmp_literal() function This omits the finding of matching provisions and only checks the package itself against the provided dep. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 78b63ce7c30f4a20707bce61eeaef913b513f557 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Aug 17 23:52:28 2011 -0500 Replacements refactor: extract check_replacers() This moves code that was inline in alpm_sync_sysupgrade() to its own method. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 231d6cc1e4083e5e49063b22fe6e5b0318b85328 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Aug 17 23:39:41 2011 -0500 Replacements refactor: extract check_literal() This moves code that was inline in alpm_sync_sysupgrade() to its own method. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5d291d050ebd714851c6bd85efd49a03d88414f0 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Aug 18 00:28:53 2011 -0500 Remove usages of alpm_list_next() in backend Another function call that can be replaced by a single pointer dereference. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c4bd476ad13e142fe8323fe74d84b3950b53da17 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Aug 17 23:25:19 2011 -0500 Remove use of no-op accessor functions in library The functions alpm_db_get_name(), alpm_pkg_get_name(), and alpm_pkg_get_version() are not necessary at all, so remove the calling and indirection when used in the backend, which makes things slightly more efficient and reduces code size. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c885a953eb888004f0302ed3eceafef93e2f072f Author: Dan McGee <dan(a)archlinux.org> Date: Wed Aug 17 21:06:04 2011 -0500 Enhance and utilize database status flags * Move is_local standalone field to status enum * Create VALID/INVALID flag pair * Create EXISTS/MISSING flag pair With these additional fields, we can be more intelligent with database loading and messages to the user. We now only warn once if a sync database does not exist and do not continue to try to load it once we have marked it as missing. The reason for the flags existing in pairs is so the unknown case can be represented. There should never be a time when both flags in the same group are true, but if they are both false, it represents the unknown case. Care is taken to always manipulate both flags at the same time. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 4a7f3bbc469d1f6a8da1c7f310ab518ad841c2b9 Author: Sebastien Luttringer <seblu(a)seblu.net> Date: Thu Aug 18 12:46:19 2011 +0200 Add makepkg -S which is an alias to makepkg --source makepkg --source is a often used go make source package like for AUR. Have a -S shortcut will save the world. Signed-off-by: Sebastien Luttringer <seblu(a)seblu.net> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d18e600952015378c9ba79661597f1c01046f436 Author: Lukas Fleischer <archlinux(a)cryptocrack.de> Date: Thu Aug 18 09:42:20 2011 +0200 lib/libalpm/dload.c: Use STRDUP() instead of strdup() Use the STRDUP macro instead of strdup() for the sake of better error handling on memory allocation failures. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 0745288c1441c73a0c6ce4279823d8363bcdc62f Author: Lukas Fleischer <archlinux(a)cryptocrack.de> Date: Thu Aug 18 09:42:21 2011 +0200 lib/libalpm/dload.c: Add ASSERT() to alpm_fetch_pkgurl() Return with ALPM_ERR_WRONG_ARGS instead of causing a potential segfault if alpm_fetch_pkgurl() is invoked with a NULL URL. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: contrib/bash_completion.in | 2 +- doc/PKGBUILD.5.txt | 21 ++-- doc/makepkg.8.txt | 14 ++-- lib/libalpm/add.c | 34 +++---- lib/libalpm/be_local.c | 25 ++++- lib/libalpm/be_sync.c | 48 ++++++--- lib/libalpm/conflict.c | 5 +- lib/libalpm/db.c | 16 ++- lib/libalpm/db.h | 18 ++-- lib/libalpm/deps.c | 40 ++++--- lib/libalpm/deps.h | 1 + lib/libalpm/diskspace.c | 6 +- lib/libalpm/dload.c | 9 +- lib/libalpm/package.c | 21 ++-- lib/libalpm/remove.c | 6 +- lib/libalpm/sync.c | 224 ++++++++++++++++++++++----------------- lib/libalpm/trans.c | 4 +- lib/libalpm/util.c | 4 +- scripts/makepkg.sh.in | 9 +- test/pacman/tests/replace103.py | 2 - 20 files changed, 299 insertions(+), 210 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] makepkg: deal with variable substitution when checking sanity
by Allan McRae 18 Aug '11

18 Aug '11

If any of pkgver, pkgrel or epoch contained a variable substitution, then it needed to be evaluated before checking its value conformed to the rules. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index e73fdfa..966caed 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1451,6 +1451,7 @@ check_sanity() { awk -F'=' '/^[[:space:]]*pkgver=/ { $1=""; print $0 }' "$BUILDFILE" | while read i _; do + eval i=$i if [[ $i =~ [[:space:]:-] ]]; then error "$(gettext "%s is not allowed to contain colons, hyphens or whitespace.")" "pkgver" return 1 @@ -1459,6 +1460,7 @@ check_sanity() { awk -F'=' '/^[[:space:]]*pkgrel=/ { $1=""; print $0 }' "$BUILDFILE" | while read i _; do + eval i=$i if [[ $i =~ [[:space:]-] ]]; then error "$(gettext "%s is not allowed to contain hyphens or whitespace.")" "pkgrel" return 1 @@ -1467,6 +1469,7 @@ check_sanity() { awk -F'=' '/^[[:space:]]*epoch=/ { $1=""; print $0 }' "$BUILDFILE" | while read i _; do + eval i=$i if [[ ! $i =~ ^[0-9]*$ ]]; then error "$(gettext "%s must be an integer.")" "epoch" return 1 -- 1.7.6

2 1

[pacman-dev] [PATCH] Add makepkg -S which is and alias to makepkg --source
by Sebastien Luttringer 18 Aug '11

18 Aug '11

makepkg --source is a often used go make source package like for AUR. Have a -S shortcut will save the world. Signed-off-by: Sebastien Luttringer <seblu(a)seblu.net> --- contrib/bash_completion.in | 2 +- doc/makepkg.8.txt | 2 +- scripts/makepkg.sh.in | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/contrib/bash_completion.in b/contrib/bash_completion.in index af5de17..8fa3da6 100644 --- a/contrib/bash_completion.in +++ b/contrib/bash_completion.in @@ -35,7 +35,7 @@ _makepkg() { opts=('allsource asroot check clean config force geninteg help holdver ignorearch install log nobuild nocheck nocolor noconfirm nodeps noextract noprogressbar nosign pkg repackage rmdeps sign skipinteg source syncdeps' - 'A L R c d e f g h i m o p r s') + 'A L R S c d e f g h i m o p r s') _arch_ptr2comp opts fi true diff --git a/doc/makepkg.8.txt b/doc/makepkg.8.txt index 3206a90..ca5a273 100644 --- a/doc/makepkg.8.txt +++ b/doc/makepkg.8.txt @@ -145,7 +145,7 @@ Options such as a chroot or remote builder. It will also satisfy requirements of the GPL when distributing binary packages. -*\--source*:: +*-S, \--source*:: Do not actually build the package, but build a source-only tarball that does not include sources that can be fetched via a download URL. This is useful for passing a single tarball to another program such as a chroot, diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index e73fdfa..8555db3 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1844,6 +1844,7 @@ usage() { echo "$(gettext " -r, --rmdeps Remove installed dependencies after a successful build")" echo "$(gettext " -R, --repackage Repackage contents of the package without rebuilding")" printf "$(gettext " -s, --syncdeps Install missing dependencies with %s")\n" "pacman" + echo "$(gettext " -S, --source Generate a source-only tarball without downloaded sources")" echo "$(gettext " --allsource Generate a source-only tarball including downloaded sources")" printf "$(gettext " --asroot Allow %s to run as root user")\n" "makepkg" printf "$(gettext " --check Run the %s function in the %s")\n" "check()" "$BUILDSCRIPT" @@ -1857,7 +1858,6 @@ usage() { echo "$(gettext " --skipchecksums Do not verify checksums of the source files")" echo "$(gettext " --skipinteg Do not perform any verification checks on source files")" echo "$(gettext " --skippgpcheck Do not verify source files with PGP signatures")" - echo "$(gettext " --source Generate a source-only tarball without downloaded sources")" echo printf "$(gettext "These options can be passed to %s:")\n" "pacman" echo @@ -1889,7 +1889,7 @@ fi ARGLIST=("$@") # Parse Command Line Options. -OPT_SHORT="AcdefFghiLmop:rRsV" +OPT_SHORT="AcdefFghiLmop:rRsSV" OPT_LONG="allsource,asroot,ignorearch,check,clean,nodeps" OPT_LONG+=",noextract,force,forcever:,geninteg,help,holdver,skippgpcheck" OPT_LONG+=",install,key:,log,nocolor,nobuild,nocheck,nosign,pkg:,rmdeps" @@ -1940,8 +1940,8 @@ while true; do --skipinteg) SKIPCHECKSUMS=1; SKIPPGPCHECK=1 ;; --skippgpcheck) SKIPPGPCHECK=1;; --sign) SIGNPKG='y' ;; - --source) SOURCEONLY=1 ;; -s|--syncdeps) DEP_BIN=1 ;; + -S|--source) SOURCEONLY=1 ;; -h|--help) usage; exit 0 ;; # E_OK -V|--version) version; exit 0 ;; # E_OK -- Sebastien "Seblu" Luttringer

2 1

[pacman-dev] [PATCH 1/6] Replacements refactor: extract check_literal()
by Dan McGee 18 Aug '11

18 Aug '11

This moves code that was inline in alpm_sync_sysupgrade() to its own method. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- lib/libalpm/sync.c | 79 ++++++++++++++++++++++++++++++--------------------- 1 files changed, 46 insertions(+), 33 deletions(-) diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index b9cb1fb..09800d4 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -81,6 +81,45 @@ alpm_pkg_t SYMEXPORT *alpm_sync_newversion(alpm_pkg_t *pkg, alpm_list_t *dbs_syn return NULL; } +static int check_literal(alpm_handle_t *handle, alpm_pkg_t *lpkg, + alpm_pkg_t *spkg, int enable_downgrade) +{ + /* 1. literal was found in sdb */ + int cmp = _alpm_pkg_compare_versions(spkg, lpkg); + if(cmp > 0) { + _alpm_log(handle, ALPM_LOG_DEBUG, "new version of '%s' found (%s => %s)\n", + lpkg->name, lpkg->version, spkg->version); + /* check IgnorePkg/IgnoreGroup */ + if(_alpm_pkg_should_ignore(handle, spkg) + || _alpm_pkg_should_ignore(handle, lpkg)) { + _alpm_log(handle, ALPM_LOG_WARNING, _("%s: ignoring package upgrade (%s => %s)\n"), + lpkg->name, lpkg->version, spkg->version); + } else { + _alpm_log(handle, ALPM_LOG_DEBUG, "adding package %s-%s to the transaction targets\n", + spkg->name, spkg->version); + return 1; + } + } else if(cmp < 0) { + if(enable_downgrade) { + /* check IgnorePkg/IgnoreGroup */ + if(_alpm_pkg_should_ignore(handle, spkg) + || _alpm_pkg_should_ignore(handle, lpkg)) { + _alpm_log(handle, ALPM_LOG_WARNING, _("%s: ignoring package downgrade (%s => %s)\n"), + lpkg->name, lpkg->version, spkg->version); + } else { + _alpm_log(handle, ALPM_LOG_WARNING, _("%s: downgrading from version %s to version %s\n"), + lpkg->name, lpkg->version, spkg->version); + return 1; + } + } else { + alpm_db_t *sdb = alpm_pkg_get_db(spkg); + _alpm_log(handle, ALPM_LOG_WARNING, _("%s: local (%s) is newer than %s (%s)\n"), + lpkg->name, lpkg->version, sdb->treename, spkg->version); + } + } + return 0; +} + /** Search for packages to upgrade and add them to the transaction. */ int SYMEXPORT alpm_sync_sysupgrade(alpm_handle_t *handle, int enable_downgrade) { @@ -101,50 +140,24 @@ int SYMEXPORT alpm_sync_sysupgrade(alpm_handle_t *handle, int enable_downgrade) continue; } - /* Search for literal then replacers in each sync database. - * If found, don't check other databases */ + /* Search for literal then replacers in each sync database. */ for(j = handle->dbs_sync; j; j = j->next) { alpm_db_t *sdb = j->data; /* Check sdb */ alpm_pkg_t *spkg = _alpm_db_get_pkgfromcache(sdb, lpkg->name); + int literal_upgrade = 0; if(spkg) { - /* 1. literal was found in sdb */ - int cmp = _alpm_pkg_compare_versions(spkg, lpkg); - if(cmp > 0) { - _alpm_log(handle, ALPM_LOG_DEBUG, "new version of '%s' found (%s => %s)\n", - lpkg->name, lpkg->version, spkg->version); - /* check IgnorePkg/IgnoreGroup */ - if(_alpm_pkg_should_ignore(handle, spkg) - || _alpm_pkg_should_ignore(handle, lpkg)) { - _alpm_log(handle, ALPM_LOG_WARNING, _("%s: ignoring package upgrade (%s => %s)\n"), - lpkg->name, lpkg->version, spkg->version); - } else { - _alpm_log(handle, ALPM_LOG_DEBUG, "adding package %s-%s to the transaction targets\n", - spkg->name, spkg->version); - trans->add = alpm_list_add(trans->add, spkg); - } - } else if(cmp < 0) { - if(enable_downgrade) { - /* check IgnorePkg/IgnoreGroup */ - if(_alpm_pkg_should_ignore(handle, spkg) - || _alpm_pkg_should_ignore(handle, lpkg)) { - _alpm_log(handle, ALPM_LOG_WARNING, _("%s: ignoring package downgrade (%s => %s)\n"), - lpkg->name, lpkg->version, spkg->version); - } else { - _alpm_log(handle, ALPM_LOG_WARNING, _("%s: downgrading from version %s to version %s\n"), - lpkg->name, lpkg->version, spkg->version); - trans->add = alpm_list_add(trans->add, spkg); - } - } else { - _alpm_log(handle, ALPM_LOG_WARNING, _("%s: local (%s) is newer than %s (%s)\n"), - lpkg->name, lpkg->version, sdb->treename, spkg->version); - } + literal_upgrade = check_literal(handle, lpkg, spkg, enable_downgrade); + if(literal_upgrade) { + trans->add = alpm_list_add(trans->add, spkg); } /* jump to next local package */ break; } else { /* 2. search for replacers in sdb */ alpm_list_t *k, *l; + _alpm_log(handle, ALPM_LOG_DEBUG, + "searching for replacements for %s\n", lpkg->name); for(k = _alpm_db_get_pkgcache(sdb); k; k = k->next) { int found = 0; spkg = k->data; -- 1.7.6

1 5

[pacman-dev] [PATCH] Reset dload_interrupted early in curl_download_internal()
by Lukas Fleischer 18 Aug '11

18 Aug '11

Avoid invoking unlink() with a NULL path if the URL isn't to a file and we fail to create a temporary file. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- lib/libalpm/dload.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 5cbb6b4..050e719 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -195,6 +195,9 @@ static int curl_download_internal(struct dload_payload *payload, /* shortcut to our handle within the payload */ alpm_handle_t *handle = payload->handle; + /* reset dload_interrupted early to avoid erroneous unlink() on cleanup */ + dload_interrupted = 0; + if(!payload->filename) { payload->filename = get_filename(payload->fileurl); } @@ -289,7 +292,6 @@ static int curl_download_internal(struct dload_payload *payload, sigaction(SIGPIPE, NULL, &sig_pipe[OLD]); sigaction(SIGPIPE, &sig_pipe[NEW], NULL); - dload_interrupted = 0; sig_int[NEW].sa_handler = &inthandler; sigemptyset(&sig_int[NEW].sa_mask); sig_int[NEW].sa_flags = 0; -- 1.7.6

2 3

[pacman-dev] [PATCH 1/2] Remove use of no-op accessor functions in library
by Dan McGee 18 Aug '11

18 Aug '11

The functions alpm_db_get_name(), alpm_pkg_get_name(), and alpm_pkg_get_version() are not necessary at all, so remove the calling and indirection when used in the backend, which makes things slightly more efficient and reduces code size. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- lib/libalpm/add.c | 34 +++++++++++++++------------------- lib/libalpm/be_local.c | 5 ++--- lib/libalpm/conflict.c | 5 ++--- lib/libalpm/db.c | 8 ++++---- lib/libalpm/package.c | 5 ++--- lib/libalpm/remove.c | 6 +++--- lib/libalpm/sync.c | 13 ++++++------- lib/libalpm/trans.c | 4 ++-- 8 files changed, 36 insertions(+), 44 deletions(-) diff --git a/lib/libalpm/add.c b/lib/libalpm/add.c index 5e1bb8d..8bbf157 100644 --- a/lib/libalpm/add.c +++ b/lib/libalpm/add.c @@ -76,8 +76,8 @@ int SYMEXPORT alpm_add_pkg(alpm_handle_t *handle, alpm_pkg_t *pkg) local = _alpm_db_get_pkgfromcache(handle->db_local, pkgname); if(local) { - const char *localpkgname = alpm_pkg_get_name(local); - const char *localpkgver = alpm_pkg_get_version(local); + const char *localpkgname = local->name; + const char *localpkgver = local->version; int cmp = _alpm_pkg_compare_versions(pkg, local); if(cmp == 0) { @@ -460,8 +460,7 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, ASSERT(trans != NULL, return -1); snprintf(scriptlet, PATH_MAX, "%s%s-%s/install", - _alpm_db_path(db), alpm_pkg_get_name(newpkg), - alpm_pkg_get_version(newpkg)); + _alpm_db_path(db), newpkg->name, newpkg->version); /* see if this is an upgrade. if so, remove the old package first */ alpm_pkg_t *local = _alpm_db_get_pkgfromcache(db, newpkg->name); @@ -517,7 +516,7 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, changelog/install script installation (FS#12263) */ if(_alpm_local_db_prepare(db, newpkg)) { alpm_logaction(handle, "error: could not create database entry %s-%s\n", - alpm_pkg_get_name(newpkg), alpm_pkg_get_version(newpkg)); + newpkg->name, newpkg->version); handle->pm_errno = ALPM_ERR_DB_WRITE; ret = -1; goto cleanup; @@ -566,10 +565,10 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, /* call PROGRESS once with 0 percent, as we sort-of skip that here */ if(is_upgrade) { PROGRESS(trans, ALPM_TRANS_PROGRESS_UPGRADE_START, - alpm_pkg_get_name(newpkg), 0, pkg_count, pkg_current); + newpkg->name, 0, pkg_count, pkg_current); } else { PROGRESS(trans, ALPM_TRANS_PROGRESS_ADD_START, - alpm_pkg_get_name(newpkg), 0, pkg_count, pkg_current); + newpkg->name, 0, pkg_count, pkg_current); } for(i = 0; archive_read_next_header(archive, &entry) == ARCHIVE_OK; i++) { @@ -593,12 +592,10 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, if(is_upgrade) { PROGRESS(trans, ALPM_TRANS_PROGRESS_UPGRADE_START, - alpm_pkg_get_name(newpkg), percent, pkg_count, - pkg_current); + newpkg->name, percent, pkg_count, pkg_current); } else { PROGRESS(trans, ALPM_TRANS_PROGRESS_ADD_START, - alpm_pkg_get_name(newpkg), percent, pkg_count, - pkg_current); + newpkg->name, percent, pkg_count, pkg_current); } /* extract the next file from the archive */ @@ -635,9 +632,9 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, if(_alpm_local_db_write(db, newpkg, INFRQ_ALL)) { _alpm_log(handle, ALPM_LOG_ERROR, _("could not update database entry %s-%s\n"), - alpm_pkg_get_name(newpkg), alpm_pkg_get_version(newpkg)); + newpkg->name, newpkg->version); alpm_logaction(handle, "error: could not update database entry %s-%s\n", - alpm_pkg_get_name(newpkg), alpm_pkg_get_version(newpkg)); + newpkg->name, newpkg->version); handle->pm_errno = ALPM_ERR_DB_WRITE; ret = -1; goto cleanup; @@ -645,15 +642,15 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, if(_alpm_db_add_pkgincache(db, newpkg) == -1) { _alpm_log(handle, ALPM_LOG_ERROR, _("could not add entry '%s' in cache\n"), - alpm_pkg_get_name(newpkg)); + newpkg->name); } if(is_upgrade) { PROGRESS(trans, ALPM_TRANS_PROGRESS_UPGRADE_START, - alpm_pkg_get_name(newpkg), 100, pkg_count, pkg_current); + newpkg->name, 100, pkg_count, pkg_current); } else { PROGRESS(trans, ALPM_TRANS_PROGRESS_ADD_START, - alpm_pkg_get_name(newpkg), 100, pkg_count, pkg_current); + newpkg->name, 100, pkg_count, pkg_current); } /* run the post-install script if it exists */ @@ -661,11 +658,10 @@ static int commit_single_pkg(alpm_handle_t *handle, alpm_pkg_t *newpkg, && !(trans->flags & ALPM_TRANS_FLAG_NOSCRIPTLET)) { if(is_upgrade) { _alpm_runscriptlet(handle, scriptlet, "post_upgrade", - alpm_pkg_get_version(newpkg), - oldpkg ? alpm_pkg_get_version(oldpkg) : NULL); + newpkg->version, oldpkg ? oldpkg->version : NULL); } else { _alpm_runscriptlet(handle, scriptlet, "post_install", - alpm_pkg_get_version(newpkg), NULL); + newpkg->version, NULL); } } diff --git a/lib/libalpm/be_local.c b/lib/libalpm/be_local.c index 5b69f62..1b48922 100644 --- a/lib/libalpm/be_local.c +++ b/lib/libalpm/be_local.c @@ -192,11 +192,10 @@ static alpm_list_t *_cache_get_backup(alpm_pkg_t *pkg) static void *_cache_changelog_open(alpm_pkg_t *pkg) { char clfile[PATH_MAX]; + alpm_db_t *db = alpm_pkg_get_db(pkg); snprintf(clfile, PATH_MAX, "%s/%s/%s-%s/changelog", alpm_option_get_dbpath(pkg->handle), - alpm_db_get_name(alpm_pkg_get_db(pkg)), - alpm_pkg_get_name(pkg), - alpm_pkg_get_version(pkg)); + db->treename, pkg->name, pkg->version); return fopen(clfile, "r"); } diff --git a/lib/libalpm/conflict.c b/lib/libalpm/conflict.c index fe18209..5d27953 100644 --- a/lib/libalpm/conflict.c +++ b/lib/libalpm/conflict.c @@ -409,7 +409,7 @@ alpm_list_t *_alpm_db_find_fileconflicts(alpm_handle_t *handle, numtargs, current); /* CHECK 1: check every target against every target */ _alpm_log(handle, ALPM_LOG_DEBUG, "searching for file conflicts: %s\n", - alpm_pkg_get_name(p1)); + p1->name); for(j = i->next; j; j = j->next) { alpm_list_t *common_files; alpm_pkg_t *p2 = j->data; @@ -422,8 +422,7 @@ alpm_list_t *_alpm_db_find_fileconflicts(alpm_handle_t *handle, for(k = common_files; k; k = k->next) { snprintf(path, PATH_MAX, "%s%s", handle->root, (char *)k->data); conflicts = add_fileconflict(handle, conflicts, - ALPM_FILECONFLICT_TARGET, path, - alpm_pkg_get_name(p1), alpm_pkg_get_name(p2)); + ALPM_FILECONFLICT_TARGET, path, p1->name, p2->name); if(handle->pm_errno == ALPM_ERR_MEMORY) { FREELIST(conflicts); FREELIST(common_files); diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c index 8a235e2..77bdf98 100644 --- a/lib/libalpm/db.c +++ b/lib/libalpm/db.c @@ -411,7 +411,7 @@ alpm_list_t *_alpm_db_search(alpm_db_t *db, const alpm_list_t *needles) for(j = list; j; j = j->next) { alpm_pkg_t *pkg = j->data; const char *matched = NULL; - const char *name = alpm_pkg_get_name(pkg); + const char *name = pkg->name; const char *desc = alpm_pkg_get_desc(pkg); /* check name as regex AND as plain text */ @@ -559,7 +559,7 @@ int _alpm_db_add_pkgincache(alpm_db_t *db, alpm_pkg_t *pkg) } _alpm_log(db->handle, ALPM_LOG_DEBUG, "adding entry '%s' in '%s' cache\n", - alpm_pkg_get_name(newpkg), db->treename); + newpkg->name, db->treename); db->pkgcache = _alpm_pkghash_add_sorted(db->pkgcache, newpkg); free_groupcache(db); @@ -576,13 +576,13 @@ int _alpm_db_remove_pkgfromcache(alpm_db_t *db, alpm_pkg_t *pkg) } _alpm_log(db->handle, ALPM_LOG_DEBUG, "removing entry '%s' from '%s' cache\n", - alpm_pkg_get_name(pkg), db->treename); + pkg->name, db->treename); db->pkgcache = _alpm_pkghash_remove(db->pkgcache, pkg, &data); if(data == NULL) { /* package not found */ _alpm_log(db->handle, ALPM_LOG_DEBUG, "cannot remove entry '%s' from '%s' cache: not found\n", - alpm_pkg_get_name(pkg), db->treename); + pkg->name, db->treename); return -1; } diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c index a88af5e..a31effb 100644 --- a/lib/libalpm/package.c +++ b/lib/libalpm/package.c @@ -618,8 +618,7 @@ void _alpm_pkg_free_trans(alpm_pkg_t *pkg) /* Is spkg an upgrade for localpkg? */ int _alpm_pkg_compare_versions(alpm_pkg_t *spkg, alpm_pkg_t *localpkg) { - return alpm_pkg_vercmp(alpm_pkg_get_version(spkg), - alpm_pkg_get_version(localpkg)); + return alpm_pkg_vercmp(spkg->version, localpkg->version); } /* Helper function for comparing packages @@ -677,7 +676,7 @@ int _alpm_pkg_should_ignore(alpm_handle_t *handle, alpm_pkg_t *pkg) alpm_list_t *groups = NULL; /* first see if the package is ignored */ - if(alpm_list_find_str(handle->ignorepkg, alpm_pkg_get_name(pkg))) { + if(alpm_list_find_str(handle->ignorepkg, pkg->name)) { return 1; } diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c index 0afa265..c6886c6 100644 --- a/lib/libalpm/remove.c +++ b/lib/libalpm/remove.c @@ -81,9 +81,9 @@ static void remove_prepare_cascade(alpm_handle_t *handle, alpm_list_t *lp) alpm_depmissing_t *miss = i->data; alpm_pkg_t *info = _alpm_db_get_pkgfromcache(handle->db_local, miss->target); if(info) { - if(!_alpm_pkg_find(trans->remove, alpm_pkg_get_name(info))) { + if(!_alpm_pkg_find(trans->remove, info->name)) { _alpm_log(handle, ALPM_LOG_DEBUG, "pulling %s in target list\n", - alpm_pkg_get_name(info)); + info->name); trans->remove = alpm_list_add(trans->remove, _alpm_pkg_dup(info)); } } else { @@ -117,7 +117,7 @@ static void remove_prepare_keep_needed(alpm_handle_t *handle, alpm_list_t *lp) pkg = vpkg; if(pkg) { _alpm_log(handle, ALPM_LOG_WARNING, _("removing %s from target list\n"), - alpm_pkg_get_name(pkg)); + pkg->name); _alpm_pkg_free(pkg); } } diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index a5964b9..b9cb1fb 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -62,20 +62,19 @@ alpm_pkg_t SYMEXPORT *alpm_sync_newversion(alpm_pkg_t *pkg, alpm_list_t *dbs_syn pkg->handle->pm_errno = 0; for(i = dbs_sync; !spkg && i; i = i->next) { - spkg = _alpm_db_get_pkgfromcache(i->data, alpm_pkg_get_name(pkg)); + spkg = _alpm_db_get_pkgfromcache(i->data, pkg->name); } if(spkg == NULL) { _alpm_log(pkg->handle, ALPM_LOG_DEBUG, "'%s' not found in sync db => no upgrade\n", - alpm_pkg_get_name(pkg)); + pkg->name); return NULL; } /* compare versions and see if spkg is an upgrade */ if(_alpm_pkg_compare_versions(spkg, pkg) > 0) { _alpm_log(pkg->handle, ALPM_LOG_DEBUG, "new version of '%s' found (%s => %s)\n", - alpm_pkg_get_name(pkg), alpm_pkg_get_version(pkg), - alpm_pkg_get_version(spkg)); + pkg->name, pkg->version, spkg->version); return spkg; } /* spkg is not an upgrade */ @@ -229,7 +228,7 @@ alpm_list_t SYMEXPORT *alpm_find_group_pkgs(alpm_list_t *dbs, for(j = grp->packages; j; j = j->next) { alpm_pkg_t *pkg = j->data; - if(_alpm_pkg_find(ignorelist, alpm_pkg_get_name(pkg))) { + if(_alpm_pkg_find(ignorelist, pkg->name)) { continue; } if(_alpm_pkg_should_ignore(db->handle, pkg)) { @@ -240,7 +239,7 @@ alpm_list_t SYMEXPORT *alpm_find_group_pkgs(alpm_list_t *dbs, if(!install) continue; } - if(!_alpm_pkg_find(pkgs, alpm_pkg_get_name(pkg))) { + if(!_alpm_pkg_find(pkgs, pkg->name)) { pkgs = alpm_list_add(pkgs, pkg); } } @@ -296,7 +295,7 @@ static int compute_download_size(alpm_pkg_t *newpkg) } _alpm_log(handle, ALPM_LOG_DEBUG, "setting download size %jd for pkg %s\n", - (intmax_t)size, alpm_pkg_get_name(newpkg)); + (intmax_t)size, newpkg->name); newpkg->infolevel |= INFRQ_DSIZE; newpkg->download_size = size; diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index 1bab830..3f59596 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -91,8 +91,8 @@ static alpm_list_t *check_arch(alpm_handle_t *handle, alpm_list_t *pkgs) const char *pkgarch = alpm_pkg_get_arch(pkg); if(pkgarch && strcmp(pkgarch, arch) && strcmp(pkgarch, "any")) { char *string; - const char *pkgname = alpm_pkg_get_name(pkg); - const char *pkgver = alpm_pkg_get_version(pkg); + const char *pkgname = pkg->name; + const char *pkgver = pkg->version; size_t len = strlen(pkgname) + strlen(pkgver) + strlen(pkgarch) + 3; MALLOC(string, len, RET_ERR(handle, ALPM_ERR_MEMORY, invalid)); sprintf(string, "%s-%s-%s", pkgname, pkgver, pkgarch); -- 1.7.6

1 1

[pacman-dev] [PATCH 1/2] lib/libalpm/dload.c: Use STRDUP() instead of strdup()
by Lukas Fleischer 18 Aug '11

18 Aug '11

Use the STRDUP macro instead of strdup() for the sake of better error handling on memory allocation failures. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- Should we do some cleanup and free() payload if STRDUP() fails in alpm_fetch_pkgurl()? lib/libalpm/dload.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 68a68e9..a1b6364 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -387,10 +387,10 @@ cleanup: tempfile, destfile, strerror(errno)); ret = -1; } else if(final_file) { - *final_file = strdup(strrchr(destfile, '/') + 1); + STRDUP(*final_file, strrchr(destfile, '/') + 1, RET_ERR(handle, ALPM_ERR_MEMORY, -1)); } } else if(final_file) { - *final_file = strdup(strrchr(tempfile, '/') + 1); + STRDUP(*final_file, strrchr(tempfile, '/') + 1, RET_ERR(handle, ALPM_ERR_MEMORY, -1)); } } @@ -456,7 +456,7 @@ char SYMEXPORT *alpm_fetch_pkgurl(alpm_handle_t *handle, const char *url) CALLOC(payload, 1, sizeof(*payload), RET_ERR(handle, ALPM_ERR_MEMORY, NULL)); payload->handle = handle; - payload->fileurl = strdup(url); + STRDUP(payload->fileurl, url, RET_ERR(handle, ALPM_ERR_MEMORY, NULL)); payload->allow_resume = 1; /* download the file */ -- 1.7.6

2 3

[pacman-dev] [PATCH 1/2] Avoid stat() on NULL path in curl_download_internal()
by Lukas Fleischer 18 Aug '11

18 Aug '11

stat()'s behaviour is undefined if the first argument is NULL and might be prone to segfault. Add an additional check to skip the stat() invocation if no destfile is used. Signed-off-by: Lukas Fleischer <archlinux(a)cryptocrack.de> --- lib/libalpm/dload.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 5a63e48..731d807 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -260,7 +260,7 @@ static int curl_download_internal(struct dload_payload *payload, curl_easy_setopt(handle->curl, CURLOPT_USERAGENT, useragent); } - if(!payload->allow_resume && !payload->force && stat(destfile, &st) == 0) { + if(!payload->allow_resume && !payload->force && destfile && stat(destfile, &st) == 0) { /* start from scratch, but only download if our local is out of date. */ curl_easy_setopt(handle->curl, CURLOPT_TIMECONDITION, CURL_TIMECOND_IFMODSINCE); curl_easy_setopt(handle->curl, CURLOPT_TIMEVALUE, (long)st.st_mtime); -- 1.7.6

2 12