diff options
author | Chantry Xavier <shiningxc@gmail.com> | 2008-02-13 20:42:43 +0100 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2008-04-26 11:54:47 -0500 |
commit | 481c3edc89fa674f9723c13ab4d12f032cf8ca02 (patch) | |
tree | 7f80c30d87878864720c45768711b0888566cede /lib/libalpm/sync.c | |
parent | 8fdf08ef782d234efe7957873839433ced64ffcc (diff) | |
download | pacman-481c3edc89fa674f9723c13ab4d12f032cf8ca02.tar.xz |
get_filename : use the FILENAME db field only.
Reference : FS#9547.
The get_filename function first tries to get the filename field from the
database, and if it doesn't find it, it tries to guess it based on the name,
version and arch.
This field was introduced in 3.0, but there are still many old entries in
the official databases without it. So the databases need to be regenerated
first before this patch can be applied.
There is a second problem with the delta code, which needs the filename for
locally installed packages too, but this field is not present in the local
db. So the delta code needs to be fixed first.
Signed-off-by: Chantry Xavier <shiningxc@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/sync.c')
-rw-r--r-- | lib/libalpm/sync.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 8dc02e06..9d087d7e 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -819,6 +819,7 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data) const char *fname = NULL; fname = alpm_pkg_get_filename(spkg); + ASSERT(fname != NULL, RET_ERR(PM_ERR_PKG_INVALID_NAME, -1)); if(trans->flags & PM_TRANS_FLAG_PRINTURIS) { EVENT(trans, PM_TRANS_EVT_PRINTURI, (char *)alpm_db_get_url(current), (char *)fname); @@ -985,6 +986,9 @@ int _alpm_sync_commit(pmtrans_t *trans, pmdb_t *db_local, alpm_list_t **data) char *fpath; fname = alpm_pkg_get_filename(spkg); + if(fname == NULL) { + goto error; + } /* Loop through the cache dirs until we find a matching file */ fpath = _alpm_filecache_find(fname); |