X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=sbr%2Futils.c;h=ef0ee9738676d018b9c191f3d339cf89593b04ff;hp=2744581c766a10ac286e8afd0a68a538bf2afc07;hb=eb650f703c26232c0ccce00c9b3dd3f6b6d51dd4;hpb=b78007de9802005825390bee71dfce2306b42519

diff --git a/sbr/utils.c b/sbr/utils.c
index 2744581..ef0ee97 100644
--- a/sbr/utils.c
+++ b/sbr/utils.c
@@ -65,6 +65,18 @@ mh_xcalloc(size_t nmemb, size_t size)
 }
 
 /*
+** Free a pointer and set it to NULL.
+*/
+void
+mh_free0(void * ptr)
+{
+	void ** p;
+	p = ptr;
+	free(*p);
+	*p = NULL;
+}
+
+/*
 ** Return the present working directory, if the current directory does not
 ** exist, or is too long, make / the pwd.
 */
@@ -95,14 +107,14 @@ pwd(void)
 /*
 ** add   -- If "s1" is NULL, this routine just creates a
 **       -- copy of "s2" into newly malloc'ed memory.
-**       -- (use getcpy() instead in this case)
+**       -- (use mh_xstrdup() instead in this case)
 **       --
 **       -- If "s1" is not NULL, then copy the concatenation
 **       -- of "s1" and "s2" (note the order) into newly
 **       -- malloc'ed memory.  Then free "s1".
 */
 char *
-add(char *s2, char *s1)
+add(const char *s2, char *s1)
 {
 	char *cp;
 	size_t len1 = 0, len2 = 0;
@@ -119,7 +131,7 @@ add(char *s2, char *s1)
 	/* Copy s1 and free it */
 	if (s1) {
 		memcpy(cp, s1, len1);
-		free(s1);
+		mh_free0(&s1);
 	}
 
 	/* Copy s2 */
@@ -155,7 +167,7 @@ create_folder(char *folder, int autocreate, void (*done_callback)(int))
 			if (!getanswer(cp)) {
 				done_callback(EX_CANTCREAT);
 			}
-			free(cp);
+			mh_free0(&cp);
 		} else if (autocreate == -1) {
 			/* do not create, so exit */
 			done_callback(EX_CANTCREAT);
@@ -206,3 +218,19 @@ app_msgarg(struct msgs_array *msgs, char *cp)
 	}
 	msgs->msgs[msgs->size++] = cp;
 }
+
+/*
+** mh_xstrdup() is a wrapper of strdup() to replace getcpy().  It returns
+** a copy of its argument if this is nonnull; otherwise, it returns a
+** string of length 0.
+*/
+char *
+mh_xstrdup(char * s)
+{
+	char * tmp;
+	tmp = strdup(s ? s : "");
+	if (!tmp) {
+		adios(EX_OSERR, "strdup", "can't copy string");
+	}
+	return tmp;
+}