anno: Stop segmentation violation on `-number 0'.

[nmh] / sbr / context_replace.c

diff --git a/sbr/context_replace.c b/sbr/context_replace.c
index 2cdadf0b9b387514f0a346da40d9a44299a4e0b9..99f8ef3a512f644182e1337afe96826bea2a127e 100644 (file)
--- a/sbr/context_replace.c
+++ b/sbr/context_replace.c
@@ -2,8 +2,6 @@
 /*
  * context_replace.c -- add/replace an entry in the context/profile list
  *
 /*
  * context_replace.c -- add/replace an entry in the context/profile list
  *

- * $Id$
- *

  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
  * complete copyright information.
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
  * complete copyright information.


@@ -16,15 +14,14 @@
 void
 context_replace (char *key, char *value)
 {
 void
 context_replace (char *key, char *value)
 {

-    register struct node *np;
+    struct node *np;

 
     /*
 
     /*

-     * If list is emtpy, allocate head of profile/context list.
+     * If list is empty, allocate head of profile/context list.

      */
     if (!m_defs) {
      */
     if (!m_defs) {

-       m_defs = (struct node *) mh_xmalloc (sizeof(*np));
-
-       np = m_defs;
+       NEW(np);
+        m_defs = np;

        np->n_name = getcpy (key);
        np->n_field = getcpy (value);
        np->n_context = 1;
        np->n_name = getcpy (key);
        np->n_field = getcpy (value);
        np->n_context = 1;


@@ -38,13 +35,12 @@ context_replace (char *key, char *value)
      * this key, and replace its value if found.
      */
     for (np = m_defs;; np = np->n_next) {
      * this key, and replace its value if found.
      */
     for (np = m_defs;; np = np->n_next) {

-       if (!strcasecmp (np->n_name, key)) {
+       if (!strcasecmp (np->n_name ? np->n_name : "", key ? key : "")) {

            if (strcmp (value, np->n_field)) {
                if (!np->n_context)
                    admonish (NULL, "bug: context_replace(key=\"%s\",value=\"%s\")", key, value);
            if (strcmp (value, np->n_field)) {
                if (!np->n_context)
                    admonish (NULL, "bug: context_replace(key=\"%s\",value=\"%s\")", key, value);

-               if (np->n_field)
-                   free (np->n_field);
-               np->n_field = getcpy (value);
+                mh_xfree(np->n_field);
+               np->n_field = mh_xstrdup(value);

                ctxflags |= CTXMOD;
            }
            return;
                ctxflags |= CTXMOD;
            }
            return;


@@ -56,8 +52,7 @@ context_replace (char *key, char *value)
     /*
      * Else add this new entry at the end
      */
     /*
      * Else add this new entry at the end
      */

-    np->n_next = (struct node *) mh_xmalloc (sizeof(*np));
-
+    NEW(np->n_next);

     np = np->n_next;
     np->n_name = getcpy (key);
     np->n_field = getcpy (value);
     np = np->n_next;
     np->n_name = getcpy (key);
     np->n_field = getcpy (value);