commit af9b59e6a1a36997d7017f4841f4a934ca1ade98
author Rosen Penev <rosenp@gmail.com> Mon Dec 21 23:24:11 2020 +0100
committer Nikias Bassen <nikias@gmx.li> Mon Jan 25 16:20:57 2021 +0100
tree f5eb124cbbfbd49b0f6195257752b62176a520ed
parent 7b1ccb403ad284b896a4e710fab578e397f101c0

Replace malloc + memset with calloc where appropriate

calloc is faster for big allocations. It's also simpler.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

libcnary/node.c

diff --git a/libcnary/node.c b/libcnary/node.c
index c24ca7a..6d68f6e 100644
--- a/libcnary/node.c
+++ b/libcnary/node.c
@@ -46,11 +46,10 @@
 node_t* node_create(node_t* parent, void* data) {
 	int error = 0;
 
-	node_t* node = (node_t*) malloc(sizeof(node_t));
-	if(node == NULL) {
+	node_t* node = (node_t*)calloc(1, sizeof(node_t));
+	if (node == NULL) {
 		return NULL;
 	}
-	memset(node, '\0', sizeof(node_t));
 
 	node->data = data;
 	node->next = NULL;

libcnary/node_list.c

diff --git a/libcnary/node_list.c b/libcnary/node_list.c
index d071881..aee3bd6 100644
--- a/libcnary/node_list.c
+++ b/libcnary/node_list.c
@@ -33,11 +33,10 @@
 }
 
 node_list_t* node_list_create() {
-	node_list_t* list = (node_list_t*) malloc(sizeof(node_list_t));
-	if(list == NULL) {
+	node_list_t* list = (node_list_t*)calloc(1, sizeof(node_list_t));
+	if (list == NULL) {
 		return NULL;
 	}
-	memset(list, '\0', sizeof(node_list_t));
 
 	// Initialize structure
 	list->begin = NULL;

src/xplist.c

diff --git a/src/xplist.c b/src/xplist.c
index 3a92142..2f98983 100644
--- a/src/xplist.c
+++ b/src/xplist.c
@@ -211,8 +211,7 @@
             struct TM _btime;
             struct TM *btime = gmtime64_r(&timev, &_btime);
             if (btime) {
-                val = (char*)malloc(24);
-                memset(val, 0, 24);
+                val = (char*)calloc(1, 24);
                 struct tm _tmcopy;
                 copy_TM64_to_tm(btime, &_tmcopy);
                 val_len = strftime(val, 24, "%Y-%m-%dT%H:%M:%SZ", &_tmcopy);

tools/plistutil.c

diff --git a/tools/plistutil.c b/tools/plistutil.c
index 989db35..fd0847b 100644
--- a/tools/plistutil.c
+++ b/tools/plistutil.c
@@ -69,8 +69,7 @@
 {
     int i = 0;
 
-    options_t *options = (options_t *) malloc(sizeof(options_t));
-    memset(options, 0, sizeof(options_t));
+    options_t *options = (options_t*)calloc(1, sizeof(options_t));
     options->out_fmt = 0;
 
     for (i = 1; i < argc; i++)