Maciej, Am 23.11.20 um 22:44 schrieb Maciej Zdeb: > There's a crash report on github: > https://github.com/haproxy/haproxy/issues/909#issuecomment-732241492 with a > reproducer and also stack trace from Willy. That list init is necessary > because release_http_action() is trying to iterate over and free its > elements: > http://git.haproxy.org/?p=haproxy.git;a=blob;f=src/http_act.c;h=13a5c370da861c78f21037186808ee16325ab45a;hb=HEAD#l52 > > Now I had more time to look at the code and maybe a cleaner solution is to > add an additional check to release_http_action() instead of initializing > the list (patch attached).
>From my point of view: I like that version better. It would have been the solution I would have taken as well. The patch should probably be BUG/MEDIUM instead of BUG/MINOR, though. Best regards Tim Düsterhus
