FGFS-4.1/simgear
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ASan: make copied files in log entries safe at init.

Browse Source 
Ensure that if we copy file-names of log entries while startup
logging is active, we free them, but only once startup logging is
disabled, or on shutdown. This is needed to avoid crashes when
we use file-name copying for the Qt message handler.
This commit is contained in:
James Turner 2020-09-05 10:43:01 +01:00
parent 52c0f04e6e
commit 908496d43d
1 changed files with 25 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
simgear/debug/logstream.cxx
View File

@ -139,8 +139,8 @@ public:
logTimer.stamp();
}
virtual void operator()(sgDebugClass c, sgDebugPriority p,
const char* file, int line, const std::string& message)
void operator()(sgDebugClass c, sgDebugPriority p,
const char* file, int line, const std::string& message) override
{
if (!shouldLog(c, p)) return;
@ -448,10 +448,18 @@ public:
{
std::lock_guard<std::mutex> g(m_lock);
m_startupLogging = on;
m_startupEntries.clear();
clearStartupEntriesLocked();
}
}
void clearStartupEntriesLocked()
{
std::for_each(m_startupEntries.begin(), m_startupEntries.end(), [](const LogEntry& e) {
if (e.freeFilename) free(const_cast<char*>(e.file));
});
m_startupEntries.clear();
}
void run() override
{
while (1) {
@ -474,11 +482,20 @@ public:
(*cb)(entry.debugClass, entry.debugPriority,
entry.file, entry.line, entry.message);
}
if (entry.freeFilename) {
// frrr the filename if required. For startup entries
// we wait and do this later
if (!m_startupLogging && entry.freeFilename) {
free(const_cast<char*>(entry.file));
}
} // of main thread loop
// house-keeping, avoid leak warnings if we exit before disabling
// startup logging
{
std::lock_guard<std::mutex> g(m_lock);
clearStartupEntriesLocked();
}
}
bool stop()
@ -536,9 +553,9 @@ public:
PauseThread pause(this);
m_logPriority = p;
m_logClass = c;
for (auto cb : m_consoleCallbacks) {
cb->setLogLevels(c, p);
}
for (auto cb : m_consoleCallbacks) {
cb->setLogLevels(c, p);
}
}
bool would_log( sgDebugClass c, sgDebugPriority p ) const