Fix invalid pointer arithmetic.

[nmh] / sbr / context_del.c

diff --git a/sbr/context_del.c b/sbr/context_del.c
index b38d822226680d17118d79193d7d0d99e700921d..592af0cad34e0d0e17593283274fc2007eb7ba16 100644 (file)
--- a/sbr/context_del.c
+++ b/sbr/context_del.c
@@ -1,8 +1,4 @@
-
-/*
- * context_del.c -- delete an entry from the context/profile list
- *
- * $Id$
+/* context_del.c -- delete an entry from the context/profile list
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
@@ -10,6 +6,7 @@
  */
 
 #include <h/mh.h>
+#include <h/utils.h>
 
 /*
  * Delete a key/value pair from the context/profile list.
@@ -19,20 +16,20 @@
 int
 context_del (char *key)
 {
-    register struct node *np, *pp;
+    struct node *np, *pp;
 
+    key = FENDNULL(key);
     for (np = m_defs, pp = NULL; np; pp = np, np = np->n_next) {
-       if (!strcasecmp (np->n_name, key)) {
+       if (!strcasecmp(FENDNULL(np->n_name), key)) {
            if (!np->n_context)
-               admonish (NULL, "bug: context_del(key=\"%s\")", np->n_name);
+               inform("bug: context_del(key=\"%s\"), continuing...", np->n_name);
            if (pp)
                pp->n_next = np->n_next;
            else
                m_defs = np->n_next;
            free (np->n_name);
-           if (np->n_field)
-               free (np->n_field);
-           free ((char *) np);
+            free(np->n_field);
+           free(np);
            ctxflags |= CTXMOD;
            return 0;
        }