[IMP] config: Test that the config file is readable
authorSamus CTO <cto@openerp.com>
Wed, 22 Oct 2014 09:40:09 +0000 (11:40 +0200)
committerChristophe Simonis <chs@odoo.com>
Fri, 24 Oct 2014 14:48:27 +0000 (16:48 +0200)
Fixes #3237

cherry-pick of 8e72049

openerp/tools/config.py

index 3e7fc33..4b3ba69 100644 (file)
@@ -350,8 +350,8 @@ class configmanager(object):
             "the i18n-export option cannot be used without the database (-d) option")
 
         # Check if the config file exists (-c used, but not -s)
-        die(not opt.save and opt.config and not os.path.exists(opt.config),
-            "The config file '%s' selected with -c/--config doesn't exist, "\
+        die(not opt.save and opt.config and not os.access(opt.config, os.R_OK),
+            "The config file '%s' selected with -c/--config doesn't exist or is not readable, "\
             "use -s/--save if you want to generate it"% opt.config)
 
         # place/search the config file on Win32 near the server installation