Flesh this out some more

[user/henk/code/inspircd.git] / src / helperfuncs.cpp

diff --git a/src/helperfuncs.cpp b/src/helperfuncs.cpp
index d0bffc7dc91b2a7d932f108caf8dcc5b8985ab75..0edb09084a07bc19119cfa7ab5152a6acef4f827 100644 (file)
--- a/src/helperfuncs.cpp
+++ b/src/helperfuncs.cpp
@@ -308,10 +308,29 @@ bool InspIRCd::OpenLog(char**, int)
                if (Config->logpath.empty())
                {
                        std::string path = std::string(home) + "/.inspircd";
+                       // This tries to create the ~/.inspircd. If it succeeds, then we go ahead and use it.
+                       // If it fails due to an existing target, then we use it anyway.
+                       // Either way, we make sure we can get write access to the log at this point.
                        if (!mkdir(path.c_str(), 0700) || errno == EEXIST)
                        {
                                /* Log to ~/.inspircd/ircd.log */
                                Config->logpath = path + "/startup.log";
+                               FILE* fd = fopen(Config->logpath.c_str(), "a+");
+                               if (!fd)
+                               {
+                                       // Could not get write access... Why?
+                                       if (errno == ENOTDIR)
+                                               // ~/.inspircd is not actually a directory!
+                                               printf("\nWARNING: Unable to create directory: %s (Exists and is not a directory)\n", path.c_str());
+                                       else
+                                               // Not writable for some other reason (no +w access, readonly fs, file too big, whatever).
+                                               printf("\nWARNING: No write access to %s (%s)\n", Config->logpath.c_str(), strerror(errno));
+                                       Config->logpath = "./startup.log";
+                               }
+                               else
+                               {
+                                       Config->log_file = fd;
+                               }
                        }
                        else
                        {
@@ -321,7 +340,8 @@ bool InspIRCd::OpenLog(char**, int)
                        }
                }
 
-               Config->log_file = fopen(Config->logpath.c_str(),"a+");
+               if (!Config->log_file)
+                       Config->log_file = fopen(Config->logpath.c_str(),"a+");
        }
        else
        {