summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan McGee <dan@archlinux.org>2011-09-27 16:14:17 -0500
committerDan McGee <dan@archlinux.org>2011-09-27 17:49:27 -0500
commitbf56fb6eb82b732900c32cbf375f7910ac479d98 (patch)
treeb370caa6f8f1c4f973358db0d22ca995d106ff37
parent619c3629ca61cea2c182fcb0197f9b84ba4209c3 (diff)
downloadpacman-bf56fb6eb82b732900c32cbf375f7910ac479d98.tar.xz
alpm_list: use malloc instead of calloc
In every case we were calling calloc, the struct we allocated (or the memory to be used) is fully specified later in the method. For alpm_list_t allocations, we always set all of data, next, and prev. For list copying and transforming to an array, we always copy the entire data element, so no need to zero it first. Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--lib/libalpm/alpm_list.c8
-rw-r--r--lib/libalpm/pkghash.c8
2 files changed, 8 insertions, 8 deletions
diff --git a/lib/libalpm/alpm_list.c b/lib/libalpm/alpm_list.c
index 274d3afd..15286aa1 100644
--- a/lib/libalpm/alpm_list.c
+++ b/lib/libalpm/alpm_list.c
@@ -89,7 +89,7 @@ alpm_list_t SYMEXPORT *alpm_list_add(alpm_list_t *list, void *data)
{
alpm_list_t *ptr, *lp;
- ptr = calloc(1, sizeof(alpm_list_t));
+ ptr = malloc(sizeof(alpm_list_t));
if(ptr == NULL) {
return list;
}
@@ -127,7 +127,7 @@ alpm_list_t SYMEXPORT *alpm_list_add_sorted(alpm_list_t *list, void *data, alpm_
} else {
alpm_list_t *add = NULL, *prev = NULL, *next = list;
- add = calloc(1, sizeof(alpm_list_t));
+ add = malloc(sizeof(alpm_list_t));
if(add == NULL) {
return list;
}
@@ -470,7 +470,7 @@ alpm_list_t SYMEXPORT *alpm_list_copy_data(const alpm_list_t *list,
const alpm_list_t *lp = list;
alpm_list_t *newlist = NULL;
while(lp) {
- void *newdata = calloc(1, size);
+ void *newdata = malloc(size);
if(newdata) {
memcpy(newdata, lp->data, size);
newlist = alpm_list_add(newlist, newdata);
@@ -775,7 +775,7 @@ void SYMEXPORT *alpm_list_to_array(const alpm_list_t *list, size_t n,
return NULL;
}
- array = calloc(n, size);
+ array = malloc(n * size);
if(array == NULL) {
return NULL;
}
diff --git a/lib/libalpm/pkghash.c b/lib/libalpm/pkghash.c
index f6207ada..963ba25e 100644
--- a/lib/libalpm/pkghash.c
+++ b/lib/libalpm/pkghash.c
@@ -159,19 +159,19 @@ static alpm_pkghash_t *pkghash_add_pkg(alpm_pkghash_t *hash, alpm_pkg_t *pkg, in
position = get_hash_position(pkg->name_hash, hash);
- ptr = calloc(1, sizeof(alpm_list_t));
+ ptr = malloc(sizeof(alpm_list_t));
if(ptr == NULL) {
return hash;
}
ptr->data = pkg;
- ptr->next = NULL;
ptr->prev = ptr;
+ ptr->next = NULL;
hash->hash_table[position] = ptr;
- if(!sorted){
+ if(!sorted) {
hash->list = alpm_list_join(hash->list, ptr);
- }else{
+ } else {
hash->list = alpm_list_mmerge(hash->list, ptr, _alpm_pkg_cmp);
}