Simplify read delimiter to use empty string in fuzz harness loop · gitpython-developers/GitPython@d79c176 · GitHub

1. gitpython-developers
2. GitPython

Repository navigation

• Code
• Issues1 (1)
• Pull requests
• Discussions
• Actions
• Security and quality42 (42)
• Insights

Commit d79c176

Browse filesBrowse the repository at this point in the historyBrowse files

DaveLak

committed

Simplify read delimiter to use empty string in fuzz harness loop

Replaces the null character delimiter `-d $'\0'` with the simpler empty string `-d ''` in the fuzzing harness build loop. This changes leverages the Bash `read` builtin behavior to avoid unnecessary complexity and improving script readability. Based @EliahKagan's suggestion and feedback on: #1901

1 parent 25f3600 commit d79c176
Copy full SHA for d79c176

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• fuzzing/oss-fuzz-scripts
  • build.sh

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎fuzzing/oss-fuzz-scripts/build.sh‎

Copy file name to clipboardExpand all lines: fuzzing/oss-fuzz-scripts/build.sh

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -13,7 +13,7 @@ find "$SEED_DATA_DIR" \( -name '*_seed_corpus.zip' -o -name '*.options' -o -name
13	13	-exec cp {} "$OUT" \;
14	14
15	15	# Build fuzzers in $OUT.
16		- find "$SRC/gitpython/fuzzing" -name 'fuzz_*.py' -print0 | while IFS= read -r -d $'\0' fuzz_harness; do
	16	+ find "$SRC/gitpython/fuzzing" -name 'fuzz_*.py' -print0 | while IFS= read -r -d '' fuzz_harness; do
17	17	compile_python_fuzzer "$fuzz_harness"
18	18
19	19	common_base_dictionary_filename="$SEED_DATA_DIR/__base.dict"

0 commit comments

Comments

0 (0)