Skip to content

Commit c4bbd9b

Browse files
szedergitster
authored andcommitted
promisor-remote: fix xcalloc() argument order
Pass the number of elements first and their size second, as expected by xcalloc(). Patch generated with: make SPATCH_FLAGS=--recursive-includes contrib/coccinelle/xcalloc.cocci.patch Our default SPATCH_FLAGS ('--all-includes') doesn't catch this transformation by default, unless used in combination with a large-ish SPATCH_BATCH_SIZE which happens to put 'promisor-remote.c' with a file that includes 'repository.h' directly in the same batch. Signed-off-by: SZEDER Gábor <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 359da65 commit c4bbd9b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

promisor-remote.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ static void promisor_remote_init(struct repository *r)
146146
if (r->promisor_remote_config)
147147
return;
148148
config = r->promisor_remote_config =
149-
xcalloc(sizeof(*r->promisor_remote_config), 1);
149+
xcalloc(1, sizeof(*r->promisor_remote_config));
150150
config->promisors_tail = &config->promisors;
151151

152152
repo_config(r, promisor_remote_config, config);

0 commit comments

Comments
 (0)