X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Finspircd.cpp;h=34a62d9189df79867d25c058f29667a63bd6d39f;hb=7a2462b891ba055cbedbf4d94ea3d5af50afa97c;hp=7a5cfe4e8c89e61ea99af49b7fe63d5e345444cf;hpb=37b46ade4b760602c814e24adaa0f0278b4f43f4;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/inspircd.cpp b/src/inspircd.cpp
index 7a5cfe4e8..34a62d918 100644
--- a/src/inspircd.cpp
+++ b/src/inspircd.cpp
@@ -226,7 +226,7 @@ namespace
 	}
 
 	// Attempts to fork into the background.
-	bool ForkIntoBackground()
+	void ForkIntoBackground()
 	{
 #ifndef _WIN32
 		// We use VoidSignalHandler whilst forking to avoid breaking daemon scripts
@@ -238,7 +238,8 @@ namespace
 		if (childpid < 0)
 		{
 			ServerInstance->Logs->Log("STARTUP", LOG_DEFAULT, "fork() failed: %s", strerror(errno));
-			return false;
+			std::cout << con_red << "Error:" << con_reset << " unable to fork into background: " << strerror(errno);
+			ServerInstance->Exit(EXIT_STATUS_FORK);
 		}
 		else if (childpid > 0)
 		{
@@ -257,7 +258,6 @@ namespace
 			SocketEngine::RecoverFromFork();
 		}
 #endif
-		return true;
 	}
 
 	// Increase the size of a core dump file to improve debugging problems.
@@ -358,6 +358,31 @@ namespace
 		signal(SIGTERM, InspIRCd::SetSignal);
 	}
 
+	void TryBindPorts()
+	{
+		FailedPortList pl;
+		ServerInstance->BindPorts(pl);
+
+		if (!pl.empty())
+		{
+			std::cout << con_red << "Warning!" << con_reset << " Some of your listener" << (pl.size() == 1 ? "s" : "") << " failed to bind:" << std::endl
+				<< std::endl;
+
+			for (FailedPortList::const_iterator iter = pl.begin(); iter != pl.end(); ++iter)
+			{
+				const FailedPort& fp = *iter;
+				std::cout << "  " << con_bright << fp.sa.str() << con_reset << ": " << strerror(fp.error) << '.' << std::endl
+					<< "  " << "Created from <bind> tag at " << fp.tag->getTagLocation() << std::endl
+					<< std::endl;
+			}
+
+			std::cout << con_bright << "Hints:" << con_reset << std::endl
+				<< "- For TCP/IP listeners try using a public IP address in <bind:address> instead" << std::endl
+				<< "  of * of leaving it blank." << std::endl
+				<< "- For UNIX socket listeners try enabling <bind:rewrite> to replace old sockets." << std::endl;
+		}
+	}
+
 	// Required for returning the proper value of EXIT_SUCCESS for the parent process.
 	void VoidSignalHandler(int)
 	{
@@ -504,13 +529,8 @@ InspIRCd::InspIRCd(int argc, char** argv)
 	SetSignals();
 	if (!Config->cmdline.runasroot)
 		CheckRoot();
-
-	if (!Config->cmdline.nofork && !ForkIntoBackground())
-	{
-		std::cout << "ERROR: could not go into daemon mode. Shutting down." << std::endl;
-		Logs->Log("STARTUP", LOG_DEFAULT, "ERROR: could not go into daemon mode. Shutting down.");
-		Exit(EXIT_STATUS_FORK);
-	}
+	if (!Config->cmdline.nofork)
+		ForkIntoBackground();
 
 	std::cout << "InspIRCd Process ID: " << con_green << getpid() << con_reset << std::endl;
 
@@ -534,30 +554,15 @@ InspIRCd::InspIRCd(int argc, char** argv)
 	// This is needed as all new XLines are marked pending until ApplyLines() is called
 	this->XLines->ApplyLines();
 
-	FailedPortList pl;
-	int bounditems = BindPorts(pl);
-
 	std::cout << std::endl;
 
+	TryBindPorts();
+
 	this->Modules->LoadAll();
 
 	// Build ISupport as ModuleManager::LoadAll() does not do it
 	this->ISupport.Build();
 
-	if (!pl.empty())
-	{
-		std::cout << std::endl << "WARNING: Not all your client ports could be bound -- " << std::endl << "starting anyway with " << bounditems
-			<< " of " << bounditems + (int)pl.size() << " client ports bound." << std::endl << std::endl;
-		std::cout << "The following port(s) failed to bind:" << std::endl << std::endl;
-		int j = 1;
-		for (FailedPortList::iterator i = pl.begin(); i != pl.end(); i++, j++)
-		{
-			std::cout << j << ".\tAddress: " << i->first.str() << " \tReason: " << strerror(i->second) << std::endl;
-		}
-
-		std::cout << std::endl << "Hint: Try using a public IP instead of blank or *" << std::endl;
-	}
-
 	std::cout << "InspIRCd is now running as '" << Config->ServerName << "'[" << Config->GetSID() << "] with " << SocketEngine::GetMaxFds() << " max open sockets" << std::endl;
 
 #ifndef _WIN32