git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 04/15] commit.c: rearrange xcalloc arguments

From
Brian Gesiak <modocache@gmail.com>
Date
May 26, 2014, 15:33 UTC
Message-ID
<1401118436-66090-5-git-send-email-modocache@gmail.com>
In-Reply-To
<1401118436-66090-1-git-send-email-modocache@gmail.com>

xcalloc takes two arguments: the number of elements and their size. reduce_heads passes the arguments in reverse order, passing the size of a commit*, followed by the number of commit* to be allocated. Rearrgange them so they are in the correct order.

Signed-off-by: Brian Gesiak <modocache@gmail.com>
---
 commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/commit.c b/commit.c
index 6bf4fe0..0fe685f 100644
--- a/commit.c
+++ b/commit.c
@@ -1041,7 +1041,7 @@ struct commit_list *reduce_heads(struct commit_list *heads)
 		p->item->object.flags |= STALE;
 		num_head++;
 	}
-	array = xcalloc(sizeof(*array), num_head);
+	array = xcalloc(num_head, sizeof(*array));
 	for (p = heads, i = 0; p; p = p->next) {
 		if (p->item->object.flags & STALE) {
 			array[i++] = p->item;
-- 
2.0.0.rc1.543.gc8042da
Previous: Brian GesiakNext: Brian Gesiak
Message 14 of 28 in “Rearrange xcalloc arguments”
  1. 00/15 Rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  2. 01/15 builtin/add.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  3. Jeremiah MahlerMay 26, 2014
  4. Brian GesiakMay 27, 2014
  5. Jeremiah MahlerMay 27, 2014
  6. Eric SunshineMay 27, 2014
  7. Brian GesiakMay 27, 2014
  8. Eric SunshineMay 27, 2014
  9. Junio C HamanoMay 27, 2014
  10. Jeremiah MahlerMay 28, 2014
  11. Brian GesiakMay 28, 2014
  12. 02/15 builtin/ls-remote.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  13. 03/15 builtin/remote.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  14. 04/15 commit.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  15. 05/15 config.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  16. 06/15 diff.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  17. 07/15 hash.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  18. 08/15 hash.h: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  19. 09/15 http-push.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  20. 10/15 imap-send.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  21. Jeremiah MahlerJun 10, 2014
  22. 11/15 notes.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  23. 12/15 pack-revindex.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  24. 13/15 reflog-walk.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  25. 14/15 remote.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  26. 15/15 transport-helper.c: rearrange xcalloc argumentsBrian Gesiak, May 26, 2014
  27. Jeremiah MahlerMay 26, 2014
  28. Jeff KingMay 28, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.