libdtsvc: Resolve coverity warnings related to 'dereference before null check' relate...

[oweals/cde.git] / cde / lib / DtSvc / DtUtil1 / Action.c

diff --git a/cde/lib/DtSvc/DtUtil1/Action.c b/cde/lib/DtSvc/DtUtil1/Action.c
index 5e3e90f6abd0bd2c8861e733dd467ef4e4d707a1..0e54643408587a0021d5142df12630966241da16 100644 (file)
--- a/cde/lib/DtSvc/DtUtil1/Action.c
+++ b/cde/lib/DtSvc/DtUtil1/Action.c
@@ -1091,7 +1091,7 @@ NoActionError(
 
    _DtCreateErrorDialog(w, actionName, msg);
    XmStringFree(msg);
-   if (msgbuf) XtFree(msgbuf);
+   XtFree(msgbuf);
    XtFree(name);
 }
 
@@ -1158,7 +1158,7 @@ TmpFileCreateError( Widget w, char *actionName, char *dirName)
        msg  = XmStringCreateLocalized(msgbuf);
         _DtCreateErrorDialog(w, actionName, msg);
         XmStringFree(msg);
-        if (msgbuf) XtFree(msgbuf);
+        XtFree(msgbuf);
 }
 
 static void
@@ -1173,7 +1173,7 @@ TmpFileOpenError( Widget w, char *actionName, char *fileName)
        msg  = XmStringCreateLocalized(msgbuf);
         _DtCreateErrorDialog(w, actionName, msg);
         XmStringFree(msg);
-        if (msgbuf) XtFree(msgbuf);
+        XtFree(msgbuf);
 }
 
 static void
@@ -1188,7 +1188,7 @@ TmpFileWriteError( Widget w, char *actionName, char *fileName)
        msg  = XmStringCreateLocalized(msgbuf);
         _DtCreateErrorDialog(w, actionName, msg);
         XmStringFree(msg);
-        if (msgbuf) XtFree(msgbuf);
+        XtFree(msgbuf);
 }
 
 static void
@@ -1202,7 +1202,7 @@ UnSupportedObject( Widget w, char *actionName, int objClass)
        msg  = XmStringCreateLocalized(msgbuf);
         _DtCreateErrorDialog(w, actionName, msg);
         XmStringFree(msg);
-        if (msgbuf) XtFree(msgbuf);
+        XtFree(msgbuf);
 }