ClearFontStuff: avoid NULL pointer dereference

Clears warning from cppcheck:

psgeom.c:400:5: warning: Either the condition 'stuff' is redundant or there
 is possible null pointer dereference: stuff. [nullPointerRedundantCheck]
    bzero(stuff, sizeof(FontStuff));
    ^
psgeom.c:398:9: note: Assuming that condition 'stuff' is not redundant
    if (stuff && stuff->foundry)
        ^
psgeom.c:400:5: note: Null pointer dereference
    bzero(stuff, sizeof(FontStuff));
    ^

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/app/xkbprint/-/merge_requests/6>

1 files changed, 3 insertions, 2 deletions

diff --git a/psgeom.c b/psgeom.c
index 33d3534..627d401 100644
--- a/psgeom.c
+++ b/psgeom.c
@@ -395,9 +395,10 @@ typedef struct {
 static void
 ClearFontStuff(FontStuff *stuff)
 {
-    if (stuff && stuff->foundry)
+    if (stuff) {
         free(stuff->foundry);
-    bzero(stuff, sizeof(FontStuff));
+        bzero(stuff, sizeof(FontStuff));
+    }
     return;
 }