mirrors/git
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

commit-graph: close descriptors after mmap

Browse Source 
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>
This commit is contained in:
Jeff King
2020-04-23 15:41:13 -06:00
committed by Junio C Hamano
parent b78a556a6a
commit c8828530b7
3 changed files with 8 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
fuzz-commit-graph.c
View File

@ -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);