diff options
author | John Keeping | 2022-02-13 15:34:50 +0000 |
---|---|---|
committer | June McEnroe | 2022-02-18 00:16:05 +0000 |
commit | 400fd7abbc9cf104999afeb05a7a40b8d58814b2 (patch) | |
tree | cd22901af7bc9cdc634543b1f13da840ae21f146 /ui-log.c | |
parent | Reset font size for blame oid (diff) | |
download | cgit-400fd7abbc9cf104999afeb05a7a40b8d58814b2.tar.gz cgit-400fd7abbc9cf104999afeb05a7a40b8d58814b2.zip |
Use release_commit_memory()
Instead of calling two separate Git functions to free memory associated with a commit object, use Git's wrapper which does this. This also counts as a potential future bug fix since release_commit_memory() also resets the parsed state of the commit, meaning any attempt to use it in the future will correctly fill out the fields again. release_commit_memory() does not set parents to zero, so keep that for additional safety in case CGit checks this without calling parse_commit() again. Signed-off-by: John Keeping <john@keeping.me.uk>
Diffstat (limited to 'ui-log.c')
-rw-r--r-- | ui-log.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/ui-log.c b/ui-log.c index b443ca7..82476e0 100644 --- a/ui-log.c +++ b/ui-log.c | |||
@@ -493,8 +493,7 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern | |||
493 | for (i = 0; i < ofs && (commit = get_revision(&rev)) != NULL; /* nop */) { | 493 | for (i = 0; i < ofs && (commit = get_revision(&rev)) != NULL; /* nop */) { |
494 | if (show_commit(commit, &rev)) | 494 | if (show_commit(commit, &rev)) |
495 | i++; | 495 | i++; |
496 | free_commit_buffer(the_repository->parsed_objects, commit); | 496 | release_commit_memory(the_repository->parsed_objects, commit); |
497 | free_commit_list(commit->parents); | ||
498 | commit->parents = NULL; | 497 | commit->parents = NULL; |
499 | } | 498 | } |
500 | 499 | ||
@@ -515,8 +514,7 @@ void cgit_print_log(const char *tip, int ofs, int cnt, char *grep, char *pattern | |||
515 | i++; | 514 | i++; |
516 | print_commit(commit, &rev); | 515 | print_commit(commit, &rev); |
517 | } | 516 | } |
518 | free_commit_buffer(the_repository->parsed_objects, commit); | 517 | release_commit_memory(the_repository->parsed_objects, commit); |
519 | free_commit_list(commit->parents); | ||
520 | commit->parents = NULL; | 518 | commit->parents = NULL; |
521 | } | 519 | } |
522 | if (pager) { | 520 | if (pager) { |