diff options
| author | Tobias Stoeckmann <tobias@stoeckmann.org> | 2016-06-06 20:12:30 +0200 |
|---|---|---|
| committer | Allan McRae <allan@archlinux.org> | 2016-08-30 10:10:40 +0200 |
| commit | 681509fd445ed6012e6ecf89b49e9c00d83b70cd (patch) | |
| tree | b2ffc59587c3277d79b70b8bd11abed994f22dd6 /lib/libalpm/backup.c | |
| parent | 1291c04961e6c27e93ef376583d46ec2aa3036a1 (diff) | |
| download | pacman-681509fd445ed6012e6ecf89b49e9c00d83b70cd.tar.gz pacman-681509fd445ed6012e6ecf89b49e9c00d83b70cd.tar.xz | |
Release resources on error paths.
Some resources (memory or file descriptors) are not released on all
error paths.
Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/backup.c')
| -rw-r--r-- | lib/libalpm/backup.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/libalpm/backup.c b/lib/libalpm/backup.c index f6225898..50bad5e6 100644 --- a/lib/libalpm/backup.c +++ b/lib/libalpm/backup.c @@ -48,9 +48,10 @@ int _alpm_split_backup(const char *string, alpm_backup_t **backup) ptr++; /* now str points to the filename and ptr points to the hash */ STRDUP((*backup)->name, str, FREE(str); return -1); - STRDUP((*backup)->hash, ptr, FREE(str); return -1); + STRDUP((*backup)->hash, ptr, FREE((*backup)->name); FREE(str); return -1); FREE(str); - return 0;} + return 0; +} /* Look for a filename in a alpm_pkg_t.backup list. If we find it, * then we return the full backup entry. |