diff --git a/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C b/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
index 0b3c8e5a7..0a7066776 100644
--- a/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
+++ b/cde/programs/dtmail/dtmail/AntiCheckBoxUiItem.C
@@ -99,7 +99,7 @@ void AntiCheckBoxUiItem::writeFromSourceToUi()
   else if (value == NULL || strcmp(value, "") == 0)
     options_checkbox_set_value(w, FALSE, this->dirty_bit);
   
-  if (NULL != NULL)
+  if (value != NULL)
     free((void*) value);
 }
 
diff --git a/cde/programs/dtmail/dtmail/CheckBoxUiItem.C b/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
index c2651b422..468a41bc7 100644
--- a/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
+++ b/cde/programs/dtmail/dtmail/CheckBoxUiItem.C
@@ -97,7 +97,7 @@ void CheckBoxUiItem::writeFromSourceToUi()
   else if (NULL == value || strcmp(value, "f") == 0)
     options_checkbox_set_value(w, FALSE, this->dirty_bit);
   
-  if (NULL != NULL)
+  if (value != NULL)
     free((void*) value);
 }
 
diff --git a/cde/programs/dtmail/dtmail/ComposeCmds.C b/cde/programs/dtmail/dtmail/ComposeCmds.C
index d3117b359..0a6d695d0 100644
--- a/cde/programs/dtmail/dtmail/ComposeCmds.C
+++ b/cde/programs/dtmail/dtmail/ComposeCmds.C
@@ -279,7 +279,7 @@ ComposeFamily::appendSignature(SendMsgDialog * compose)
     compose->get_editor()->textEditor()->append_to_contents(fullpath);
     if (NULL != fullpath)
       free((void*) fullpath);
-    if (NULL != NULL)
+    if (NULL != value)
       free((void*) value);
 
     compose->get_editor()->textEditor()->set_to_top();