Don't need to cast to `char *' for free(3) these days.

But still need a cast to discard the const.
Use `void *' as it saves needing to check the original type.

diff --git a/uip/mhfixmsg.c b/uip/mhfixmsg.c
index 7fd7caa..ee25624 100644 (file)
--- a/uip/mhfixmsg.c
+++ b/uip/mhfixmsg.c
@@ -2598,8 +2598,8 @@ fix_filename_encoding (CT ct) {
                 const char *const new_params = concat (params, "\n", NULL);
 
                 replace_substring (&hf->value, semicolon_loc, new_params);
-                free ((char *) new_params);
-                free ((char *) params);
+                free((void *)new_params); /* Cast away const.  Sigh. */
+                free((void *)params);
             } else {
                 advise (NULL, "did not find semicolon in %s:%s\n",
                         hf->name, hf->value);