On Wednesday 15 June 2011 07:56:25 Mike Blumenkrantz wrote:
> On Wed, 15 Jun 2011 07:55:27 +0200 (CEST)
> Vincent Torri <vto...@univ-evry.fr> wrote:
> > On Wed, 15 Jun 2011, Mike Blumenkrantz wrote:
> > > I like the idea of better error messages, but using access creates a
> > > permissions loophole where the file could be switched between access()
> > > and open(). You should probably check it another way.
> > 
> > ecore_file_exists ?
> > 
> > Vincent
> 
> same loophole.

..but a better choice nonetheless. I know that the MS Windows CE runtime is 
seriously castrated, so having just one central place for workarounds is 
surely beneficial. I'll adjust the patch, unless I introduce a new dependency 
on Ecore.

> I suppose it's not really that big of a deal...

No it isn't. If you wanted to do it 100% correct, you could only use open() 
and then handle errors from that, but getting sensible errors out of that is 
much harder. Further, the race condition can't be exploited for anything since 
nothing is written.

Cheers!

Uli

------------------------------------------------------------------------------
EditLive Enterprise is the world's most technically advanced content
authoring tool. Experience the power of Track Changes, Inline Image
Editing and ensure content is compliant with Accessibility Checking.
http://p.sf.net/sfu/ephox-dev2dev
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to