Suricata

Is this rather an optimization than a bug ?
Does it deserve a backport to 7 ?

In my opinion, it is a bug, as the original behavior does not make sense (using the BypassedCheckFuncInit function and checking only after that whether it exists).

Adam Kiripolsky wrote in #note-6:

In my opinion, it is a bug, as the original behavior does not make sense (using the BypassedCheckFuncInit function and checking only after that whether it exists).

So, can we get a call of a NULL-pointer callback in the current situation ? How can this happen ?

I see something that looks correct in the current code

        if (bypassedfunclist[i].FuncInit) {
            bypassedfunclist[i].FuncInit(th_v, &curtime, bypassedfunclist[i].data);
        }