Avoids to load a directory structure as a rules file

Author: Felipe Zimmerle

src/parser/driver.cc

@@ -157,7 +157,7 @@ int Driver::parseFile(const std::string &f) {
     std::ifstream t(f);
     std::string str;
 
-    if (t.is_open() == false) {
+    if (utils::isFile(f) == false) {
         m_parserError << "Failed to open the file: " << f << std::endl;
         return false;
     }

src/utils/system.cc

@@ -144,5 +144,19 @@ bool createDir(std::string dir, int mode, std::string *error) {
 }
 
 
+bool isFile(std::string f) {
+    struct stat fileInfo;
+    FILE *fp = fopen(f.c_str(), "r");
+    fstat(fileno(fp), &fileInfo);
+    if (!S_ISREG(fileInfo.st_mode)) {
+        fclose(fp);
+        return false;
+    }
+    fclose(fp);
+
+    return true;
+}
+
+
 }  // namespace utils
 }  // namespace modsecurity

src/utils/system.h

@@ -34,7 +34,7 @@ std::string find_resource(const std::string& file, const std::string& param,
 std::string get_path(const std::string& file);
 std::list<std::string> expandEnv(const std::string& var, int flags);
 bool createDir(std::string dir, int mode, std::string *error);
-
+bool isFile(std::string f);
 
 }  // namespace utils
 }  // namespace modsecurity