Hi Lei,<br><br>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.<br>
<br><div class="gmail_quote">On Fri, Oct 8, 2010 at 4:58 PM, 章磊 <span dir="ltr"><<a href="mailto:ioripolo@gmail.com">ioripolo@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi clang,<br><br>Sorry for the late reply. In this patch i re-implement the chrootchecker.<br><br>Main changes are:<br><ol><li>Remove the SymbolEnv, use enums to represent chroot() jail state.<br></li><li>Directly set the enum value in the GDM.</li>

<li>Add some comments.</li></ol>Is there any advice about this patch?<br><font color="#888888"><br><br>-- <br>Best regards!<br><br>Lei Zhang<br>
</font></blockquote></div><br>