Update to make boofuzz more compatible

[hegusung]

I had these 3 issues when using boofuzz:

• I could not change processmonitor.py listening interface
• processmonitor.py was not compatible with Windows, multiple function used in boofuzz were not available for Windows.
• Some processes to fuzz needs a lot of time before starting up, so the hardcoded 3 seconds delay were not sufficient. A parameter "restart_wait" in Session class has been added to solve this issue

[jtpereyda]

@hegusung Sounds great! I'll put this on my queue to review. Which operating systems have you tested on so far with these changes?

[hegusung]

Hi, So boofuzz script was tested on both a Kali linux and a Debian. Process_monitor.py was tested on a Windows 10.

there shouldn't be an issue with process_monitor.py on linux because the linux way is always done, il only switches to windows way when an exeption is detected.

[hegusung]

I don't understand what is wrong with stickler-ci

[SR4ven]

I don't understand what is wrong with stickler-ci

Seems like it runs into a timeout. Nothing u need to worry about.
If you want to check for code style issues, you can run tox or tox -e lint (not exactly sure about the second one). Also note that it outputs all findings, not just the ones in your changes.

[SR4ven]

Thanks for the PR @hegusung!

I left some comments below.

[jtpereyda]

@hegusung Thanks for this work! This was a significant gap in boofuzz.

After this PR, it wouldn't be much work to integrate the two procmon scripts, since the Simple debugger will work with both Windows and Unix.

Thanks @SR4ven for the review!

[hegusung]

Hi, I see that this PR is not accepted.
Which changes should I do in order for you guys to accept it ?

[hegusung]

I took care of your remarks,

I also patched the following bug:

• when the second or more mutant fails 3 times, the skipped value will be 0. therefore the test case number is incorrect.
• This is due to self.mutant_index is necessary bigger than self.fuzz_node.mutant.get_num_mutations() at the second and more fuzzed mutant

[SR4ven]

Apologies for the delayed review.

Thanks for implementing our comments!
I've added one more tiny suggestion.

Could you add some lines about those fixes to the CHANGELOG.rst please?

[SR4ven]

What do you think about mutation_index?

Suggested change

	self.index_mutation = 0
	self.mutation_index = 0