On Fri, Aug 20, 2021 at 01:56:20PM -0400, Jeff King wrote:
quoted
It would help unify the handling of code that is somewhat
duplicated (but slightly different each time) across
credential_apply_config(), http_init(), get_urlmatch(),
and cmd__urlmatch_normalization().
Agreed. It looks like http_init() has the same leak that is fixed here.
Oh, nevermind. That call is indeed correct. Apparently if you re-order
two lines I'm unable to see them. ;)
-Peff