<div dir="ltr">I just left some comments but the code was merged as I was writing them.<div><br></div><div>Casey, it looked like there was a TOCTOU race in one of the patches (<a href="https://review.tizen.org/gerrit/#/c/24123/3/provider-interface/download-provider-interface.c">https://review.tizen.org/gerrit/#/c/24123/3/provider-interface/download-provider-interface.c</a>).  Would you please confirm?<div>
<br></div><div>Thanks,</div><div>-Ossama</div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 13, 2014 at 9:40 AM, Stéphane Desneux <span dir="ltr"><<a href="mailto:stephane.desneux@open.eurogiciel.org" target="_blank">stephane.desneux@open.eurogiciel.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Casey did the review and merged the commit.<br>
<br>
I submitted it on tizen_common. everything builds fine (including<br>
tizen-extension-crosswalk). I think it's safe to accept it in the main<br>
project.<br>
<br>
Thanks for the heads-up<br>
--<br>
Stéphane Desneux<br>
Intel OTC - Vannes/FR<br>
gpg:1CA35726/DFA9B0232EF80493AF2891FA24E3A2841CA35726<br>
<div><div class="h5"><br>
On 13/08/2014 17:09, <a href="mailto:corentin.lecouvey@open.eurogiciel.org">corentin.lecouvey@open.eurogiciel.org</a> wrote:<br>
> Hi all,<br>
><br>
> A code review is required on the following patch :<br>
> <a href="https://review.tizen.org/gerrit/#/c/24123/3" target="_blank">https://review.tizen.org/gerrit/#/c/24123/3</a><br>
> This patch was proposed a month ago...<br>
><br>
> It fixes many problems (64 bits, multiuser , smack, ... ) and make<br>
> download API works over Crosswalk.<br>
><br>
> For information, it also reinstalls 'net-config' by runtime dependency.<br>
><br>
> Can anybody review this patch ?<br>
><br>
> Thanks and regards,<br>
> Corentin<br>
><br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> Dev mailing list<br>
> <a href="mailto:Dev@lists.tizen.org">Dev@lists.tizen.org</a><br>
> <a href="https://lists.tizen.org/listinfo/dev" target="_blank">https://lists.tizen.org/listinfo/dev</a><br>
><br>
_______________________________________________<br>
Dev mailing list<br>
<a href="mailto:Dev@lists.tizen.org">Dev@lists.tizen.org</a><br>
<a href="https://lists.tizen.org/listinfo/dev" target="_blank">https://lists.tizen.org/listinfo/dev</a><br>
</blockquote></div><br></div>