No need to declare variable name as static in dounsetenv(). Also, drop a

redundant call to free() since name is always freed upon return.

From Michael W. Bombardieri, with some tweaks by myself.

diff --git a/bin/csh/func.c b/bin/csh/func.c
index 1a74763..18707bf 100644 (file)
--- a/bin/csh/func.c
+++ b/bin/csh/func.c
@@ -1,4 +1,4 @@
-/*    $OpenBSD: func.c,v 1.36 2017/12/16 10:27:21 anton Exp $       */
+/*    $OpenBSD: func.c,v 1.37 2017/12/18 19:12:24 anton Exp $       */
 /*    $NetBSD: func.c,v 1.11 1996/02/09 02:28:29 christos Exp $       */
 
 /*-
@@ -924,11 +924,9 @@ void
 /*ARGSUSED*/
 dounsetenv(Char **v, struct command *t)
 {
-    Char  **ep, *p, *n;
+    Char  **ep, *p, *n, *name;
     int     i, maxi;
-    static Char *name = NULL;
 
-    free(name);
     /*
      * Find the longest environment variable
      */
@@ -957,7 +955,6 @@ dounsetenv(Char **v, struct command *t)
                break;
            }
     free(name);
-    name = NULL;
 }
 
 void