update ForkserverBytesCoverageSugar to use parse_afl_cmdline

[epi052]

Updates ForkserverBytesCoverageSugar to use parse_afl_cmdline when using ForkserverExecutorBuilder to create a ForkserverExecutor.

The current impl uses args which don't account for "@@".

[domenukk]

I think we should also use:

        .is_deferred_frksrv(true)
        .is_persistent(true)
        .autotokens(&mut tokens)

[domenukk]

Also it should use build_dynamic_map, (basically this, see

LibAFL/fuzzers/fuzzbench_forkserver/src/main.rs

Line 312 in e5b3e5a

let forkserver = ForkserverExecutor::builder()

)

[epi052]

Also it should use build_dynamic_map, (basically this, see

If we move the sugar wrapper to use build_dynamic_map, is there any use for the MAP_SIZE generic?

impl<'a, const MAP_SIZE: usize> ForkserverBytesCoverageSugar<'a, MAP_SIZE>

[domenukk]

@tokatoka pointed out that this won't work for some targets, so it's better to not use deferred forkserver by default here (afl++ solves this by scanning the binary, but we don't want to do that imho)

[domenukk]

Also it should use build_dynamic_map, (basically this, see

If we move the sugar wrapper to use build_dynamic_map, is there any use for the MAP_SIZE generic?

impl<'a, const MAP_SIZE: usize> ForkserverBytesCoverageSugar<'a, MAP_SIZE>

I didn't look into it, so not sure?

[epi052]

once all the tests run, i think this is ready for a look.

I tested it without the generic / with the build_dynamic using exercise-3 (which uses ASAN and a map size of 86217) and it worked.

[domenukk]

Looks good, thanks! <3
(sorry I was on holidays)