RepoMirrors
/
selinux-refpolicy
mirror of https://github.com/SELinuxProject/refpolicy
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fc_sort: use calloc instead of malloc 

Rather than using malloc to allocated nodes and setting all the fields,
just use calloc.

Signed-off-by: William Roberts <william.c.roberts@intel.com>
This commit is contained in:
William Roberts 2017-10-17 11:16:23 -07:00
parent 1b405f4a90
commit 65620e0f94
1 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
support/fc_sort.c
View File

@ -357,15 +357,11 @@ int main(int argc, char *argv[])
} }
/* Initialize the head of the linked list. */ /* Initialize the head of the linked list. */
head = current = (file_context_node_t*)malloc(sizeof(file_context_node_t)); head = current = (file_context_node_t*)calloc(1, sizeof(file_context_node_t));
if (!head) { if (!head) {
fprintf(stderr, "Error: failure allocating memory.\n"); fprintf(stderr, "Error: failure allocating memory.\n");
return 1; return 1;
} }
head->next = NULL;
head->path = NULL;
head->file_type = NULL;
head->context = NULL;
/* Parse the file into a file_context linked list. */ /* Parse the file into a file_context linked list. */
line_buf = NULL; line_buf = NULL;
@ -390,15 +386,13 @@ int main(int argc, char *argv[])
continue; continue;
/* We have a valid line - allocate a new node. */ /* We have a valid line - allocate a new node. */
temp = (file_context_node_t *)malloc(sizeof(file_context_node_t)); temp = (file_context_node_t *)calloc(1, sizeof(file_context_node_t));
if (!temp) { if (!temp) {
free(line_buf); free(line_buf);
fprintf(stderr, "Error: failure allocating memory.\n"); fprintf(stderr, "Error: failure allocating memory.\n");
fc_free_file_context_node_list(head); fc_free_file_context_node_list(head);
return 1; return 1;
} }
temp->next = NULL;
memset(temp, 0, sizeof(file_context_node_t));
/* Parse out the regular expression from the line. */ /* Parse out the regular expression from the line. */
start = i; start = i;