[PATCH] D37750: [llvm-objdump] Fix memory leaks in macho dump

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 25 13:52:04 PDT 2017


Ping

On Mon, Sep 18, 2017 at 11:30 AM David Blaikie via Phabricator via
llvm-commits <llvm-commits at lists.llvm.org> wrote:

> dblaikie added inline comments.
>
>
> ================
> Comment at: llvm/trunk/tools/llvm-objdump/MachODump.cpp:5884
>      outs() << "\n";
> +    xar_iter_free(xp);
>    }
> ----------------
> Any chance of an RAII API for this resource, to avoid the accident-prone
> manual cleanup/free?
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D37750
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170925/f90aa773/attachment.html>


More information about the llvm-commits mailing list