[cfe-dev] A patch for chroot checker

Zhongxing Xu xuzhongxing at gmail.com
Sat Oct 9 22:51:32 PDT 2010


Hi Lei,

I committed the patch with minor modification: int does not always has the
same width as void*. intptr_t is used. The patch looks good for now, though
I'm not quite sure about the rationale behind the check. Since the checker
does not affect other parts of the analyzer, we could improve it later.

On Fri, Oct 8, 2010 at 4:58 PM, 章磊 <ioripolo at gmail.com> wrote:

> Hi clang,
>
> Sorry for the late reply. In this patch i re-implement the chrootchecker.
>
> Main changes are:
>
>    1. Remove the SymbolEnv, use enums to represent chroot() jail state.
>    2. Directly set the enum value in the GDM.
>    3. Add some comments.
>
> Is there any advice about this patch?
>
>
> --
> Best regards!
>
> Lei Zhang
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20101010/aa57bfd7/attachment.html>


More information about the cfe-dev mailing list