AFLplusplus / AFLplusplus/LibAFL

Do not sync initial corpus, or don't do it too fast

Abierto
#2,564 3 comentarios 0 reacciones 0 asignados Ver en GitHub
enhancement
Lenguaje dominante
Rust
Estrellas
2.6k
Forks
481
Merge medio
2 d 30 min
PR fusionados (30 d)
16

Descripción

From somebody using LibAFL:

> I'm hitting this panic from time to time: https://github.com/AFLplusplus/LibAFL/blob/main/libafl_bolts/src/llmp.rs#L1114
> I think it's because I'm launching a lot of clients, that may be slow, with a big corpus.
> the constant definition says:
> ```
> /// The max number of pages a [`client`] may have mapped that were not yet read by the [`broker`]
> /// Usually, this value should not exceed `1`, else the broker cannot keep up with the amount of incoming messages.
> /// Instead of increasing this value, you may consider sending new messages at a lower rate, else your Sender will eventually `OOM`.
> const LLMP_CFG_MAX_PENDING_UNREAD_PAGES: usize = 3;
> ```

My answer to have this thing fixed quickly is this one:

> a quick and dirty patch is this one https://github.com/AFLplusplus/LibAFL/blob/main/libafl/src/state/mod.rs#L751
> change the call to evaluate_input to evaluate_input_events that takes a parameter more, send_events, set it to false

How should we address this problem? Apply my patch? We can assume all the nodes load the initial corpus by themself so no need to create NewTestcase events, but maybe there are situation I'm not thinking about now that are useful.
The initial corpus loading methods are already very configurable and we have many variants, I would avoid adding another one to let the user choose whatever send NewTestcase messages when loading the initial corpus or not and simply hardcode the best solution.

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.