if (ptr) free(ptr) is not a good idiom.

1 files changed, 1 insertions, 2 deletions

diff --git a/src/main.c b/src/main.c
index ab1f592..f8d3fed 100644
--- a/src/main.c
+++ b/src/main.c
@@ -297,8 +297,7 @@ handler (Display *d, XErrorEvent *e)
 
   /* If there is already an error to report, replace it with this new
      one. */
-  if (rp_error_msg)
-    free (rp_error_msg);
+  free (rp_error_msg);
   rp_error_msg = xstrdup (error_msg);
 
   return 0;