[Webkit-unassigned] [Bug 60361] [EFL] EWebLauncher creates only one settings databases directory at startup.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 11 23:26:31 PDT 2011


https://bugs.webkit.org/show_bug.cgi?id=60361





--- Comment #12 from Tomasz Morawski <t.morawski at samsung.com>  2011-05-11 23:26:30 PST ---
(In reply to comment #11)
> IMO it makes sense to at least check if the path is already a directory in WebCore itself. 

Lets lookat the makeAllDirectories function:

  if (!access(fullPath.data(), F_OK))
        return true;

The access(fullPath.data(), F_OK) function, returns 0 if path already exist. So, I can check there if this is a directory path and if user have rights to it... if not return false insted of true. I may try to make a patch for makeAllDirectories that will do it. But I am not sure if it will be accepted. 

> As for the permission part, if you decide to keep it in WebKit, you could use ecore_file_can_{read,exec,write}.
OK, this is a good idea. Thank you for that. I suggest to keep it if patch for makeAllDirectories will be not accepted.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list