RFR (XXS): 8075288: malloc without free in VM_PopulateDumpSharedSpace::doit()

[Kim Barrett]

On Apr 10, 2015, at 9:31 AM, Jungwoo Ha <jwha at google.com> wrote:
> 
> What's the conclusion? Should I fix it or discuss it separately?

I think the proposed fix, as is, is good.

I think the question of whether this code should be using malloc/free or something else (stack allocation, resource allocation) is a separate question.  I don’t know anything about the code or the context(s) in which it is used, so I don’t have an informed opinion on that.  I’m not sure this is really gc code; might be better to bring that up on the hs-dev list.