From 2dcc187c92c1a579e6e9f0bad999a3b4e47228c3 Mon Sep 17 00:00:00 2001 From: Matthieu Herrb Date: Sat, 18 Mar 2006 15:18:56 +0000 Subject: doublecheck that a pointer is not NULL before dereferencing it. (Coverity CID 121). --- ChangeLog | 6 ++++++ src/Attrib.c | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 20b9b3e..afae7e2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2006-03-18 Matthieu Herrb + + * src/Attrib.c: XpmFreeExtensions + doublecheck that a pointer is not NULL before dereferencing it. + (Coverity CID 121). + 2005-12-14 Kevin E. Martin * configure.ac: diff --git a/src/Attrib.c b/src/Attrib.c index c6b34d6..a6feb52 100644 --- a/src/Attrib.c +++ b/src/Attrib.c @@ -133,7 +133,7 @@ XpmFreeExtensions(extensions, nextensions) XpmFree(ext->name); nlines = ext->nlines; for (j = 0, sptr = ext->lines; j < nlines; j++, sptr++) - if (*sptr) + if (sptr && *sptr) XpmFree(*sptr); if (ext->lines) XpmFree(ext->lines); -- cgit v1.2.1