Commit af9b59e6 authored by Rosen Penev's avatar Rosen Penev Committed by Nikias Bassen

Replace malloc + memset with calloc where appropriate

calloc is faster for big allocations. It's also simpler.
Signed-off-by: 's avatarRosen Penev <rosenp@gmail.com>
parent 7b1ccb40
......@@ -46,11 +46,10 @@ void node_destroy(node_t* node) {
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;
......
......@@ -33,11 +33,10 @@ void node_list_destroy(node_list_t* list) {
}
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;
......
......@@ -211,8 +211,7 @@ static void node_to_xml(node_t* node, bytearray_t **outbuf, uint32_t depth)
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);
......
......@@ -69,8 +69,7 @@ static options_t *parse_arguments(int argc, char *argv[])
{
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++)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment