tyler.durden
/
icinga2
mirror of https://github.com/Icinga/icinga2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert "Use threads for external commands" 

This reverts commit e1b8b05ef3.
This commit is contained in:
Gunnar Beutner 2014-08-21 09:36:05 +02:00
parent 071ac754ca
commit 80762a0360
2 changed files with 11 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
components/compat/externalcommandlistener.cpp
View File

@ -113,8 +113,6 @@ void ExternalCommandListener::CommandPipeThread(const String& commandPath)
FILE *fp = fdopen(fd, "r"); FILE *fp = fdopen(fd, "r");
if (fp == NULL) { if (fp == NULL) {
(void) close(fd);
std::ostringstream msgbuf; std::ostringstream msgbuf;
msgbuf << "fdopen() for fifo path '" << commandPath << "' failed with error code " << errno << ", \"" << Utility::FormatErrorNumber(errno) << "\""; msgbuf << "fdopen() for fifo path '" << commandPath << "' failed with error code " << errno << ", \"" << Utility::FormatErrorNumber(errno) << "\"";
Log(LogCritical, "ExternalCommandListener", msgbuf.str()); Log(LogCritical, "ExternalCommandListener", msgbuf.str());
@ -130,24 +128,21 @@ void ExternalCommandListener::CommandPipeThread(const String& commandPath)
(line[strlen(line) - 1] == '\r' || line[strlen(line) - 1] == '\n')) (line[strlen(line) - 1] == '\r' || line[strlen(line) - 1] == '\n'))
line[strlen(line) - 1] = '\0'; line[strlen(line) - 1] = '\0';
Utility::QueueAsyncCallback(boost::bind(&ExternalCommandListener::ExecuteCommand, line)); String command = line;
try {
Log(LogInformation, "ExternalCommandListener", "Executing external command: " + command);
ExternalCommandProcessor::Execute(command);
} catch (const std::exception&) {
std::ostringstream msgbuf;
msgbuf << "External command failed.";
Log(LogWarning, "ExternalCommandListener", msgbuf.str());
}
} }
delete line; delete line;
fclose(fp); fclose(fp);
} }
} }
void ExternalCommandListener::ExecuteCommand(const String& command)
{
try {
Log(LogInformation, "ExternalCommandListener", "Executing external command: " + command);
ExternalCommandProcessor::Execute(command);
} catch (const std::exception&) {
std::ostringstream msgbuf;
msgbuf << "External command failed: " << command;
Log(LogWarning, "ExternalCommandListener", msgbuf.str());
}
}
#endif /* _WIN32 */ #endif /* _WIN32 */

1
components/compat/externalcommandlistener.hpp
View File

@ -49,7 +49,6 @@ private:
boost::thread m_CommandThread; boost::thread m_CommandThread;
void CommandPipeThread(const String& commandPath); void CommandPipeThread(const String& commandPath);
static void ExecuteCommand(const String& command);
#endif /* _WIN32 */ #endif /* _WIN32 */
}; };