commit 91a4400fd5a74c6e954b22b276dd38c7ffaeae33
author Eric Andersen <andersen@codepoet.org> Wed Jul 19 17:37:57 2000 +0000
committer Eric Andersen <andersen@codepoet.org> Wed Jul 19 17:37:57 2000 +0000
tree 33cf648853df478b733452c48f665e35ad65dc1f
parent a16c66335e24009c4cbcd57ce8205b4dfc7b099c

Avoid trying to free NULL ptrs.  Comment on malloc usages.
 -Erik

cmdedit.c

diff --git a/cmdedit.c b/cmdedit.c
index 0f064b4..0de18e8 100644
--- a/cmdedit.c
+++ b/cmdedit.c
@@ -370,14 +370,16 @@
 
 void get_previous_history(struct history **hp, char* command)
 {
-	free((*hp)->s);
+	if ((*hp)->s)
+		free((*hp)->s);
 	(*hp)->s = strdup(command);
 	*hp = (*hp)->p;
 }
 
 void get_next_history(struct history **hp, char* command)
 {
-	free((*hp)->s);
+	if ((*hp)->s)
+		free((*hp)->s);
 	(*hp)->s = strdup(command);
 	*hp = (*hp)->n;
 }
@@ -654,7 +656,7 @@
 		struct history *h = his_end;
 
 		if (!h) {
-			/* No previous history */
+			/* No previous history -- this memory is never freed */
 			h = his_front = malloc(sizeof(struct history));
 			h->n = malloc(sizeof(struct history));
 
@@ -666,7 +668,7 @@
 			his_end = h->n;
 			history_counter++;
 		} else {
-			/* Add a new history command */
+			/* Add a new history command -- this memory is never freed */
 			h->n = malloc(sizeof(struct history));
 
 			h->n->p = h;