varnishcache / varnish-cache

Varnish Cache source code repository
https://www.varnish-cache.org
Other
3.56k stars 366 forks source link

Call VDP_Close() for all cases and revert regression #4068

Closed nigoroll closed 4 months ago

nigoroll commented 4 months ago

This PR fixes the issues described in #4067 :

We add missing VDP_Close() calls to the deliver functions of esi and http1, revert a wrong addition of a VDP_Close() call, add one at the right place and add a VDP_Fini() function with an assertion to ensure all cases are taken care of.

nigoroll commented 4 months ago

merged as of bb3d027bf6647185be548050243e1cb9d36741a7 but it seems there is an issue with gh noticing the merge