On Wed, 15 Jun 2011 21:27:13 +0200 Ulrich Eckhardt <dooms...@knuut.de> said:
> 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. the way efl works you can't really always get sensible errors as efl likes to keep re-opening and closing files in the background and "page from them" so an error u see on initial file set or not may not match one later on that you never even can get the error code of. -- ------------- Codito, ergo sum - "I code, therefore I am" -------------- The Rasterman (Carsten Haitzler) ras...@rasterman.com ------------------------------------------------------------------------------ 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
