<div dir="ltr">On Thu, Mar 26, 2015 at 3:45 PM, Piotr Caban <span dir="ltr"><<a href="mailto:piotr.caban@gmail.com" target="_blank">piotr.caban@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<span class=""><br>
<br>
On 26/03/15 19:21, Sebastian Lackner wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
On 26.03.2015 19:10, Piotr Caban wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
The inheritance doesn't occur when function from ntdll are used for<br>
setting DACL. This makes your code inside server incorrect. Also your<br>
code is not checking handle permissions correctly.<br>
</blockquote></span>
does that mean you reviewed our code? Would you please give some more details to us about the error you found?<span class=""><br>
Feel free to open a bug report on <a href="http://bugs.wine-staging.com" target="_blank">bugs.wine-staging.com</a> with your results.<br>
</span></blockquote>
I've not really reviewed it, I've only skimmed the diff. According to my testing NtSetSecurityObject is only setting supplied DACL, it's not adding inherited entries.<br>
<br>
I also haven't seen the implementation from wine-staging tree earlier (I think these patches were not mentioned anywhere on wine bugzilla/were never sent to wine but maybe I've missed something).<br>
<br>
Thanks,<br>
Piotr<br></blockquote><div><br><a href="https://bugs.winehq.org/show_bug.cgi?id=33576#c10">https://bugs.winehq.org/show_bug.cgi?id=33576#c10</a> <br></div></div><br>-- <br><div class="gmail_signature">-Austin</div>
</div></div>