From fb285d869548a75457bf7fe89a22801de4269c91 Mon Sep 17 00:00:00 2001
From: Samus CTO <cto@openerp.com>
Date: Fri, 27 Feb 2015 17:16:16 +0100
Subject: [PATCH] [FIX] prevent unneeded chmod datadir, but check writable

---
 openerp/tools/config.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/openerp/tools/config.py b/openerp/tools/config.py
index b07bbf99c05..21b15132fa3 100644
--- a/openerp/tools/config.py
+++ b/openerp/tools/config.py
@@ -683,7 +683,8 @@ class configmanager(object):
         if not os.path.exists(d):
             os.makedirs(d, 0700)
         else:
-            os.chmod(d, 0700)
+            assert os.access(d, os.W_OK), \
+                "%s: directory is not writable" % d
         return d
 
     @property
@@ -692,7 +693,8 @@ class configmanager(object):
         if not os.path.exists(d):
             os.makedirs(d, 0700)
         else:
-            os.chmod(d, 0700)
+            assert os.access(d, os.W_OK), \
+                "%s: directory is not writable" % d
         return d
 
     def filestore(self, dbname):