Let template manager throw for configuration errors

Change-Id: Iec25cd08ae5cff6ef721b77022d07f734898f773

diff --git a/src/common/include/smack-rules.h b/src/common/include/smack-rules.h
index 35f6d8d..53e7402 100644 (file)
--- a/src/common/include/smack-rules.h
+++ b/src/common/include/smack-rules.h
@@ -38,9 +38,7 @@ namespace SecurityManager {
 
 class SmackRules {
 public:
-    SmackRules() : m_templateMgr(POLICY_INSTALL_DIR) {
-        m_templateMgr.init();
-    }
+    SmackRules();
 
     /**
      * Install package-specific smack rules plus add rules for specified external apps.

diff --git a/src/common/include/template-manager.h b/src/common/include/template-manager.h
index 6844902..bb6aba9 100644 (file)
--- a/src/common/include/template-manager.h
+++ b/src/common/include/template-manager.h
@@ -25,12 +25,20 @@
 #include <map>
 #include <string>
 
+#include <dpl/exception.h>
 #include <smack-common.h>
 
 #include "config-file.h"
 
 namespace SecurityManager {
 
+class TemplateManagerException {
+public:
+    DECLARE_EXCEPTION_TYPE(SecurityManager::Exception, Base)
+    DECLARE_EXCEPTION_TYPE(Base, ConfigFileError)
+    DECLARE_EXCEPTION_TYPE(Base, ConfigParseError)
+};
+
 class TemplateManager {
 public:
     explicit TemplateManager(const std::string &rootDir) : m_rootDir(rootDir){}

diff --git a/src/common/smack-rules.cpp b/src/common/smack-rules.cpp
index c205865..68aac2f 100644 (file)
--- a/src/common/smack-rules.cpp
+++ b/src/common/smack-rules.cpp
@@ -74,6 +74,14 @@ const std::string SMACK_APP_PATH_USER_PERMS = "rwxat";
 
 } // namespace anonymous
 
+SmackRules::SmackRules() : m_templateMgr(POLICY_INSTALL_DIR) {
+    try {
+        m_templateMgr.init();
+    } catch (TemplateManagerException::Base &e) {
+        LogError("Error loading template files: " << e.DumpToString());
+    }
+}
+
 void SmackRules::addFromTemplate(
     SmackAccesses &rules,
     TemplateManager::Type type,

diff --git a/src/common/template-manager.cpp b/src/common/template-manager.cpp
index 3f6cc09..605d9ee 100644 (file)
--- a/src/common/template-manager.cpp
+++ b/src/common/template-manager.cpp
@@ -57,8 +57,10 @@ void TemplateManager::init()
         loadFiles();
     } catch (FS::Exception::Base &e) {
         LogError("Error loading config files: " << e.DumpToString());
+        Throw(TemplateManagerException::ConfigFileError);
     } catch (SmackException::FileError &e) {
         LogError("Error parsing config files: " << e.DumpToString());
+        Throw(TemplateManagerException::ConfigParseError);
     }
 }