From b94e8ecd1fec4426baab8c90e7fc0d5583acdbef Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Tue, 28 Jun 2011 17:46:04 -0500 Subject: Fix a few warnings pointed out via clang scan-build Some of these are legit (the backup hash NULL checks), while others are either extemely unlikely or just impossible for the static code analysis to prove, but are worth adding anyway because they have little overhead. Signed-off-by: Dan McGee --- lib/libalpm/conflict.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/libalpm/conflict.c') diff --git a/lib/libalpm/conflict.c b/lib/libalpm/conflict.c index e54c7147..5d9bbf23 100644 --- a/lib/libalpm/conflict.c +++ b/lib/libalpm/conflict.c @@ -65,7 +65,7 @@ void _alpm_conflict_free(alpm_conflict_t *conflict) alpm_conflict_t *_alpm_conflict_dup(const alpm_conflict_t *conflict) { alpm_conflict_t *newconflict; - CALLOC(newconflict, 1, sizeof(alpm_conflict_t), ); + CALLOC(newconflict, 1, sizeof(alpm_conflict_t), return NULL); STRDUP(newconflict->package1, conflict->package1, return NULL); STRDUP(newconflict->package2, conflict->package2, return NULL); -- cgit v1.2.3-24-g4f1b