[Dev] Need a review on download-provider
ossama.othman at intel.com
Wed Aug 13 16:56:15 GMT 2014
I just left some comments but the code was merged as I was writing them.
Casey, it looked like there was a TOCTOU race in one of the patches (
Would you please confirm?
On Wed, Aug 13, 2014 at 9:40 AM, Stéphane Desneux <
stephane.desneux at open.eurogiciel.org> wrote:
> Casey did the review and merged the commit.
> I submitted it on tizen_common. everything builds fine (including
> tizen-extension-crosswalk). I think it's safe to accept it in the main
> Thanks for the heads-up
> Stéphane Desneux
> Intel OTC - Vannes/FR
> On 13/08/2014 17:09, corentin.lecouvey at open.eurogiciel.org wrote:
> > Hi all,
> > A code review is required on the following patch :
> > https://review.tizen.org/gerrit/#/c/24123/3
> > This patch was proposed a month ago...
> > It fixes many problems (64 bits, multiuser , smack, ... ) and make
> > download API works over Crosswalk.
> > For information, it also reinstalls 'net-config' by runtime dependency.
> > Can anybody review this patch ?
> > Thanks and regards,
> > Corentin
> > _______________________________________________
> > Dev mailing list
> > Dev at lists.tizen.org
> > https://lists.tizen.org/listinfo/dev
> Dev mailing list
> Dev at lists.tizen.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Dev