summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrew Gregory <andrew.gregory.8@gmail.com>2014-04-27 01:24:55 +0200
committerAllan McRae <allan@archlinux.org>2014-05-04 08:14:44 +0200
commitdd694f433b0848a04ae72e771086b1bcd8b9a4e9 (patch)
tree1e5b2406a5be2eb1cfd43709f0bc9d80a2981b09
parent5293d5784049869c965ee21f424b8845bdebb745 (diff)
downloadpacman-dd694f433b0848a04ae72e771086b1bcd8b9a4e9.tar.gz
pacman-dd694f433b0848a04ae72e771086b1bcd8b9a4e9.tar.xz
register_syncdb: reject db names that contain /
alpm downloads databases based on basename causing [foo] and [bar/foo] to silently overwrite each other. Also remove an extra tab Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com> Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--lib/libalpm/db.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c
index b9cbed76..466c96f1 100644
--- a/lib/libalpm/db.c
+++ b/lib/libalpm/db.c
@@ -52,12 +52,13 @@ alpm_db_t SYMEXPORT *alpm_register_syncdb(alpm_handle_t *handle,
CHECK_HANDLE(handle, return NULL);
ASSERT(treename != NULL && strlen(treename) != 0,
RET_ERR(handle, ALPM_ERR_WRONG_ARGS, NULL));
+ ASSERT(!strchr(treename, '/'), RET_ERR(handle, ALPM_ERR_WRONG_ARGS, NULL));
/* Do not register a database if a transaction is on-going */
ASSERT(handle->trans == NULL, RET_ERR(handle, ALPM_ERR_TRANS_NOT_NULL, NULL));
/* ensure database name is unique */
if(strcmp(treename, "local") == 0) {
- RET_ERR(handle, ALPM_ERR_DB_NOT_NULL, NULL);
+ RET_ERR(handle, ALPM_ERR_DB_NOT_NULL, NULL);
}
for(i = handle->dbs_sync; i; i = i->next) {
alpm_db_t *d = i->data;