On Mon, Dec 12, 2016 at 04:50:20PM +0530, Karthik Nayak wrote: > > This caller never stores the return value, and it ends up leaking. So I > > wonder if you wanted "static struct strbuf" in the first place (and that > > would explain the strbuf_reset() in your function). > > Ah! Yes this should be 'static struct strbuf' indeed, I blindly copied Junio's > suggestion. > > strbuf_detach() is also a better way to go. One of the other, though. If it's static, then you _don't_ want to detach. -Peff