[PATCH v10] git: use calloc instead of malloc + memset where possible
From: AreaZR via GitGitGadget <hidden>
Date: 2024-12-18 01:26:30
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Seija Kijin <redacted>
Avoid calling malloc + memset by calling calloc.
Signed-off-by: Seija <redacted>
---
git: use calloc instead of malloc where possible
We can avoid having to call memset by calling calloc
Signed-off-by: Seija doremylover123@gmail.com
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1390%2FAreaZR%2Fcalloc-v10
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1390/AreaZR/calloc-v10
Pull-Request: https://github.com/git/git/pull/1390
Range-diff vs v9:
1: 35994dfb8af ! 1: d7b7959be0a git: use calloc instead of malloc where possible
@@ Metadata
Author: Seija Kijin [off-list ref]
## Commit message ##
- git: use calloc instead of malloc where possible
+ git: use calloc instead of malloc + memset where possible
- Avoid having to calling malloc+memset by calling calloc.
+ Avoid calling malloc + memset by calling calloc.
Signed-off-by: Seija [off-list ref]
remote.c | 4 ++--
submodule.c | 10 +++++-----
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/remote.c b/remote.c
index 10104d11e3c..462ff105273 100644
--- a/remote.c
+++ b/remote.c@@ -2854,9 +2854,9 @@ void apply_push_cas(struct push_cas_option *cas, struct remote_state *remote_state_new(void) { - struct remote_state *r = xmalloc(sizeof(*r)); + struct remote_state *r; - memset(r, 0, sizeof(*r)); + CALLOC_ARRAY(r, 1); hashmap_init(&r->remotes_hash, remotes_hash_cmp, NULL, 0); hashmap_init(&r->branches_hash, branches_hash_cmp, NULL, 0);
diff --git a/submodule.c b/submodule.c
index 7ec564854d0..7707c6f48f0 100644
--- a/submodule.c
+++ b/submodule.c@@ -1489,14 +1489,13 @@ struct fetch_task { */ static const struct submodule *get_non_gitmodules_submodule(const char *path) { - struct submodule *ret = NULL; + struct submodule *ret; const char *name = default_name_or_path(path); if (!name) return NULL; - ret = xmalloc(sizeof(*ret)); - memset(ret, 0, sizeof(*ret)); + CALLOC_ARRAY(ret, 1); ret->path = name; ret->name = name;
@@ -1536,8 +1535,9 @@ static struct fetch_task *fetch_task_create(struct submodule_parallel_fetch *spf const char *path, const struct object_id *treeish_name) { - struct fetch_task *task = xmalloc(sizeof(*task)); - memset(task, 0, sizeof(*task)); + struct fetch_task *task; + + CALLOC_ARRAY(task, 1); if (validate_submodule_path(path) < 0) exit(128);
base-commit: 063bcebf0c917140ca0e705cbe0fdea127e90086 -- gitgitgadget