diff options
| author | Dan McGee <dan@archlinux.org> | 2011-06-29 00:46:04 +0200 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-06-30 18:51:36 +0200 |
| commit | b94e8ecd1fec4426baab8c90e7fc0d5583acdbef (patch) | |
| tree | f3d4d2800610893956ccad1691509ddf53c7f487 /lib/libalpm/backup.c | |
| parent | 9efd10cd2ac9a7654b2c7c94df86bf09ab54f41a (diff) | |
| download | pacman-b94e8ecd1fec4426baab8c90e7fc0d5583acdbef.tar.gz pacman-b94e8ecd1fec4426baab8c90e7fc0d5583acdbef.tar.xz | |
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 <dan@archlinux.org>
Diffstat (limited to 'lib/libalpm/backup.c')
| -rw-r--r-- | lib/libalpm/backup.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |