Commit7b684cd

committed

Add graceful handling of expected exceptions infuzz_submodule.py

Fixes:https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=69350**`IsADirectoryError`**Fuzzer provided input data can sometimes produce filenames that looklike directories and raise `IsADirectoryError` exceptions which crashthe fuzzer. This commit catches those cases and returns -1 to instructlibfuzzer that the inputs are not valuable to add to the corpus.**`FileExistsError`**Similar to the above, this is a possible exception case produced by thefuzzed data and not a bug so its handled the same.

1 parent2345c1a commit7b684cdCopy full SHA for 7b684cd

File tree

1 file changed

-1

lines changed

fuzzing/fuzz-targets
- fuzz_submodule.py

1 file changed

-1

lines changed

`‎fuzzing/fuzz-targets/fuzz_submodule.py`

Lines changed: 9 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,15 @@ def TestOneInput(data):`
`67`	`67`	`)`
`68`	`68`	`repo.index.commit(f"Removed submodule{submodule_name}")`
`69`	`69`
`70`		`-except (ParsingError,GitCommandError,InvalidGitRepositoryError,FileNotFoundError,BrokenPipeError):`
	`70`	`+except (`
	`71`	`+ParsingError,`
	`72`	`+GitCommandError,`
	`73`	`+InvalidGitRepositoryError,`
	`74`	`+FileNotFoundError,`
	`75`	`+FileExistsError,`
	`76`	`+IsADirectoryError,`
	`77`	`+BrokenPipeError,`
	`78`	`+ ):`
`71`	`79`	`return-1`
`72`	`80`	`except (ValueError,OSError)ase:`
`73`	`81`	`expected_messages= [`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit7b684cd

File tree

1 file changed

1 file changed

`‎fuzzing/fuzz-targets/fuzz_submodule.py`

0 commit comments