summaryrefslogtreecommitdiff
path: root/fuzz-commit-graph.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2020-04-23 21:41:13 (GMT)
committerJunio C Hamano <gitster@pobox.com>2020-04-25 05:25:50 (GMT)
commitc8828530b7797f5ab584c84dc2b86d3c14b39c8d (patch)
tree1b90f0518485d0547f6654429b1c8780cf187ece /fuzz-commit-graph.c
parentb78a556a6aef0d1406e0038f6cb8401f0282fa98 (diff)
downloadgit-c8828530b7797f5ab584c84dc2b86d3c14b39c8d.zip
git-c8828530b7797f5ab584c84dc2b86d3c14b39c8d.tar.gz
git-c8828530b7797f5ab584c84dc2b86d3c14b39c8d.tar.bz2
commit-graph: close descriptors after mmap
We don't ever refer to the descriptor after mmap-ing it. And keeping it open means we can run out of descriptors in degenerate cases (e.g., thousands of split chain files). Let's close it as soon as possible. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Taylor Blau <me@ttaylorr.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'fuzz-commit-graph.c')
-rw-r--r--fuzz-commit-graph.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/fuzz-commit-graph.c b/fuzz-commit-graph.c
index 0157acb..9fd1c04 100644
--- a/fuzz-commit-graph.c
+++ b/fuzz-commit-graph.c
@@ -1,8 +1,7 @@
#include "commit-graph.h"
#include "repository.h"
-struct commit_graph *parse_commit_graph(void *graph_map, int fd,
- size_t graph_size);
+struct commit_graph *parse_commit_graph(void *graph_map, size_t graph_size);
int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size);
@@ -11,7 +10,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size)
struct commit_graph *g;
initialize_the_repository();
- g = parse_commit_graph((void *)data, -1, size);
+ g = parse_commit_graph((void *)data, size);
repo_clear(the_repository);
free(g);