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/backup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/libalpm/backup.c') diff --git a/lib/libalpm/backup.c b/lib/libalpm/backup.c index 0334042c..becc7be9 100644 --- a/lib/libalpm/backup.c +++ b/lib/libalpm/backup.c @@ -40,7 +40,7 @@ int _alpm_split_backup(const char *string, alpm_backup_t **backup) STRDUP(str, string, return -1); /* tab delimiter */ - ptr = strchr(str, '\t'); + ptr = str ? strchr(str, '\t') : NULL; if(ptr == NULL) { (*backup)->name = str; (*backup)->hash = NULL; -- cgit v1.2.3-24-g4f1b