On Fri, Aug 20, 2021 at 01:56:20PM -0400, Jeff King wrote: > > 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