X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=sbr%2Fcrawl_folders.c;h=71646162b0f2934bc856ab2ddc78a5f16440d30c;hp=59ec95209b0ae90f3bd790b9bb49ca2847dadae2;hb=fc9279e818dfc96c63a5d75a89080cc68cfe1170;hpb=db62cda50055368e5f4ac51df787053f70706c30

diff --git a/sbr/crawl_folders.c b/sbr/crawl_folders.c
index 59ec952..7164616 100644
--- a/sbr/crawl_folders.c
+++ b/sbr/crawl_folders.c
@@ -9,6 +9,8 @@
 #include <h/mh.h>
 #include <h/crawl_folders.h>
 #include <h/utils.h>
+#include <dirent.h>
+#include <sys/stat.h>
 
 struct crawl_context {
 	int max;    /*
@@ -29,7 +31,7 @@ struct crawl_context {
 static void
 add_folder(char *fold, struct crawl_context *crawl)
 {
-	register int i, j;
+	int i, j;
 
 	/* if necessary, reallocate the space for folder names */
 	if (crawl->foldp >= crawl->max) {
@@ -66,7 +68,7 @@ add_children(char *name, struct crawl_context *crawl)
 	}
 
 	if (strcmp(name, ".") == 0) {
-		prefix = getcpy("");
+		prefix = mh_xstrdup("");
 	} else {
 		prefix = concat(name, "/", (void *)NULL);
 	}
@@ -100,12 +102,12 @@ add_children(char *name, struct crawl_context *crawl)
 			/* add_folder saves child in the list, don't free it */
 			add_folder(child, crawl);
 		} else {
-			free(child);
+			mh_free0(&child);
 		}
 	}
 
 	closedir(dd);
-	free(prefix);
+	mh_free0(&prefix);
 }
 
 static void
@@ -140,10 +142,10 @@ crawl_folders_body(struct crawl_context *crawl, char *dir,
 void
 crawl_folders(char *dir, crawl_callback_t *callback, void *baton)
 {
-	struct crawl_context *crawl = mh_xmalloc(sizeof(*crawl));
+	struct crawl_context *crawl = mh_xcalloc(1, sizeof(*crawl));
 	crawl->max = CRAWL_NUMFOLDERS;
 	crawl->total = crawl->start = crawl->foldp = 0;
-	crawl->folders = mh_xmalloc(crawl->max * sizeof(*crawl->folders));
+	crawl->folders = mh_xcalloc(crawl->max, sizeof(*crawl->folders));
 
 	crawl_folders_body(crawl, dir, callback, baton);
 
@@ -151,6 +153,6 @@ crawl_folders(char *dir, crawl_callback_t *callback, void *baton)
 	** Note that we "leak" the folder names, on the assumption that the
 	** caller is using them.
 	*/
-	free(crawl->folders);
-	free(crawl);
+	mh_free0(&(crawl->folders));
+	mh_free0(&crawl);
 }