diff options
author | Andrew Gregory <andrew.gregory.8@gmail.com> | 2015-11-01 20:41:55 -0500 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2015-11-03 21:45:09 +1000 |
commit | 6eac7258cd3ae1cb63e59fceb68cc5d144a9dd4f (patch) | |
tree | 73dbc7a429b1af1045f637b6c5ece14f8254d79a /lib/libalpm/be_sync.c | |
parent | 0fd6d354a6b8de976ec9b6a146c4cb8cdcdd5ebb (diff) | |
download | pacman-6eac7258cd3ae1cb63e59fceb68cc5d144a9dd4f.tar.xz |
ensure realloc has a positive size
If given size 0 POSIX allows realloc to return a pointer that is not
suitable for use.
Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/be_sync.c')
-rw-r--r-- | lib/libalpm/be_sync.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c index 25c8cbec..b09b0608 100644 --- a/lib/libalpm/be_sync.c +++ b/lib/libalpm/be_sync.c @@ -721,9 +721,13 @@ static int sync_db_read(alpm_db_t *db, struct archive *archive, files_count++; } /* attempt to hand back any memory we don't need */ - files = realloc(files, sizeof(alpm_file_t) * files_count); - /* make sure the list is sorted */ - qsort(files, files_count, sizeof(alpm_file_t), _alpm_files_cmp); + if(files_count > 0) { + files = realloc(files, sizeof(alpm_file_t) * files_count); + /* make sure the list is sorted */ + qsort(files, files_count, sizeof(alpm_file_t), _alpm_files_cmp); + } else { + FREE(files); + } pkg->files.count = files_count; pkg->files.files = files; } |