[pacman-dev] [PATCH v2] libalpm: clone data on alpm_db_set_servers
morganamilo
morganamilo at archlinux.org
Fri Apr 30 07:59:56 UTC 2021
Every alpm_option_set function clones the input so lets be more
consistent. Also this fixes servers not being sanatized.
---
v2: fix copy paste error
---
lib/libalpm/db.c | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c
index 8511bb83..b8d1b157 100644
--- a/lib/libalpm/db.c
+++ b/lib/libalpm/db.c
@@ -139,9 +139,15 @@ alpm_list_t SYMEXPORT *alpm_db_get_servers(const alpm_db_t *db)
int SYMEXPORT alpm_db_set_servers(alpm_db_t *db, alpm_list_t *servers)
{
+ alpm_list_t *i;
ASSERT(db != NULL, return -1);
FREELIST(db->servers);
- db->servers = servers;
+ for(i = servers; i; i = i->next) {
+ char *url = i->data;
+ if(alpm_db_add_server(db, url) != 0) {
+ return -1;
+ }
+ }
return 0;
}
--
2.31.1
More information about the pacman-dev
mailing list