tree 83424b9d6dbc082fbf698050a98404f6d6290017
parent 7e371dc9944790f18c55e12c8797fb6ba012cfc3
author Nicholas Nethercote <n.nethercote@gmail.com> 1692939713 +1000
committer Daniel Stenberg <daniel@haxx.se> 1692970031 +0200
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFDBAABCgAtFiEEJ+3q8i86vOtQ25oSXMkI/bceEsIFAmTorDoPHGRhbmllbEBo
 YXh4LnNlAAoJEFzJCP23HhLCIt0H/j0feAhuxhsXdgOG6nIuFUaAxQviRhbjVv0K
 wvqn8DfsyC0dITf0wtr/tt4sQlXp69Y4ArA2Bam1wvI3lqIrObbEREeHsMl5lT6K
 kuyZolT8SZOp/DLAqiCavC/rRtCIMvCy0LYvKtGhwirqr7RV9RXUJB0wn2PFqq1z
 W1s4FE4u2GEKlCP4x+eHcfJHsCDww/WlqEKGeziQPnsR8ppn1Tzw42aoygmwWsZW
 0WqPiKbyFQ8EX2JnOMH0pVb3YZaVe+kl2gH7mWpVMGdXr8IZoahV0zcM3yO2nFKa
 g7hHlgy4zAg9t8ajovv33lIBMheBbzYgAKEKMIJ/5Pe0JqSrESw=
 =S7ed
 -----END PGP SIGNATURE-----

c-hyper: fix a memory leak in `Curl_http`.

A request created with `hyper_request_new` must be consumed by either
`hyper_clientconn_send` or `hyper_request_free`.

This is not terrifically clear from the hyper docs --
`hyper_request_free` is documented only with "Free an HTTP request if
not going to send it on a client" -- but a perusal of the hyper code
confirms it.

This commit adds a `hyper_request_free` to the `error:` path in
`Curl_http` so that the request is consumed when an error occurs after
the request is created but before it is sent.

Fixes the first memory leak reported by Valgrind in #10803.

Closes #11729