diff options
| author | Dan McGee <dan@archlinux.org> | 2008-02-24 08:17:17 +0100 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2008-02-24 08:17:17 +0100 |
| commit | 96f7613d15e46131bf8a4b93828ad70b041524d1 (patch) | |
| tree | f65e1b56d32519fa8c85e8339746f3a5b5e9d37b /src/pacman/callback.c | |
| parent | 105e01c8efcb3a27cb55cc5dd7bf15afbea80698 (diff) | |
| download | pacman-96f7613d15e46131bf8a4b93828ad70b041524d1.tar.gz pacman-96f7613d15e46131bf8a4b93828ad70b041524d1.tar.xz | |
Add some NULL checks into recently modified output functions
After a merge with master where some strings we print (such as descriptions)
could be NULL, a few segfaults popped up due to strlen() calls on null
pointers. Fix this by doing some preemptive checks and returning from
functions early if the string was null.
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'src/pacman/callback.c')
| -rw-r--r-- | src/pacman/callback.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/pacman/callback.c b/src/pacman/callback.c index 629c2e1c..01e65a95 100644 --- a/src/pacman/callback.c +++ b/src/pacman/callback.c @@ -592,7 +592,7 @@ void cb_dl_progress(const char *filename, int file_xfered, int file_total, /* Callback to handle notifications from the library */ void cb_log(pmloglevel_t level, char *fmt, va_list args) { - if(strlen(fmt) == 0) { + if(!fmt || strlen(fmt) == 0) { return; } |