==3859==ERROR: AddressSanitizer: heap-use-after-free on address 0x61b0000053e0 at pc 0x000800697160 bp 0x7fffffffcfb0 sp 0x7fffffffcfa8 READ of size 8 at 0x61b0000053e0 thread T0 #0 0x80069715f in PT::MemBuffer<wchar_t, 256ul, 4096ul>::empty() const /usr/home/tomek/roboczy/prog/web.ttmath.org/../pikotools/membuffer/membuffer.h:675:9 #1 0x80069700d in PT::TextStreamBase<wchar_t, 256ul, 4096ul>::empty() const /usr/home/tomek/roboczy/prog/web.ttmath.org/../pikotools/textstream/textstream.h:161:16 #2 0x8009af0e3 in PT::FileLog::save_log(PT::TextStreamBase<wchar_t, 256ul, 4096ul>*) /usr/home/tomek/roboczy/prog/pikotools/log/filelog.cpp:105:14 #3 0x8009b3f8a in PT::Log::save_log() /usr/home/tomek/roboczy/prog/pikotools/log/log.cpp:461:13 #4 0x8009b3d72 in PT::Log::save_log_and_clear() /usr/home/tomek/roboczy/prog/pikotools/log/log.cpp:448:2 #5 0x8009b0627 in PT::Log::~Log() /usr/home/tomek/roboczy/prog/pikotools/log/log.cpp:62:2 #6 0x8007288e7 in Winix::Log::~Log() /usr/home/tomek/roboczy/prog/winix/winixd/core/log.cpp:56:1 #7 0x8007eac96 in Winix::WinixBase::~WinixBase() /usr/home/tomek/roboczy/prog/winix/winixd/core/winixbase.cpp:51:1 #8 0x8007eb097 in Winix::WinixModel::~WinixModel() /usr/home/tomek/roboczy/prog/winix/winixd/core/winixmodel.cpp:52:1 #9 0x80069e552 in Winix::BaseThread::~BaseThread() /usr/home/tomek/roboczy/prog/winix/winixd/core/basethread.cpp:54:1 #10 0x2f8132 in Winix::Job::~Job() /usr/home/tomek/roboczy/prog/web.ttmath.org/../winix/winixd/core/job.h:54:7 #11 0x2e0c8a in Winix::System::~System() /usr/home/tomek/roboczy/prog/web.ttmath.org/../winix/winixd/core/system.h:67:7 #12 0x2df017 in Winix::App::~App() /usr/home/tomek/roboczy/prog/web.ttmath.org/../winix/winixd/core/app.h:70:7 #13 0x800cc9f50 in __cxa_finalize /hddraidzfs/usr/src/lib/libc/stdlib/atexit.c:238:5 #14 0x800c5ef10 in exit /hddraidzfs/usr/src/lib/libc/stdlib/exit.c:74:2 #15 0x25c935 in _start /hddraidzfs/usr/src/lib/csu/amd64/crt1.c:76:2 0x61b0000053e0 is located 1632 bytes inside of 1656-byte region [0x61b000004d80,0x61b0000053f8) freed by thread T0 here: #0 0x2dc07d in operator delete(void*) /hddraidzfs/usr/src/contrib/llvm-project/compiler-rt/lib/asan/asan_new_delete.cpp:160:3 #1 0x8007c3b2c in Winix::ThreadManager::StopAll() /usr/home/tomek/roboczy/prog/winix/winixd/core/threadmanager.cpp:189:3 #2 0x800647474 in Winix::App::WaitForThreads() /usr/home/tomek/roboczy/prog/winix/winixd/core/app.cpp:2122:24 #3 0x800647311 in Winix::App::Close() /usr/home/tomek/roboczy/prog/winix/winixd/core/app.cpp:340:2 #4 0x2dec7c in main /usr/home/tomek/roboczy/prog/winix/winixd/main/main.cpp:224:6 #5 0x25c92e in _start /hddraidzfs/usr/src/lib/csu/amd64/crt1.c:76:7 #6 0x80033efff (<unknown module>) previously allocated by thread T0 here: #0 0x2db81d in operator new(unsigned long) /hddraidzfs/usr/src/contrib/llvm-project/compiler-rt/lib/asan/asan_new_delete.cpp:99:3 #1 0x8007c17c3 in Winix::ThreadManager::Add(Winix::BaseThread*, wchar_t const*) /usr/home/tomek/roboczy/prog/winix/winixd/core/threadmanager.cpp:77:26 #2 0x8007b0b81 in Winix::System::Init() /usr/home/tomek/roboczy/prog/winix/winixd/core/system.cpp:170:17 #3 0x8006468d4 in Winix::App::Init() /usr/home/tomek/roboczy/prog/winix/winixd/core/app.cpp:295:9 #4 0x2deb7e in main /usr/home/tomek/roboczy/prog/winix/winixd/main/main.cpp:206:11 #5 0x25c92e in _start /hddraidzfs/usr/src/lib/csu/amd64/crt1.c:76:7 #6 0x80033efff (<unknown module>) in ThreadManager::StopAll() there is item.thread_item_data pointer which was freed (the pointer was pointing to an object with a log_buffer) but internal Log buffer from item.object was pointing to that object and when thread_tab.clear() was called then the d-ctor was called and then save_log_and_clear() method was called and tried to use the freed object.
282 lines
5.1 KiB
C++
282 lines
5.1 KiB
C++
/*
|
|
* This file is a part of Winix
|
|
* and is distributed under the 2-Clause BSD licence.
|
|
* Author: Tomasz Sowa <t.sowa@ttmath.org>
|
|
*/
|
|
|
|
/*
|
|
* Copyright (c) 2010-2019, Tomasz Sowa
|
|
* All rights reserved.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions are met:
|
|
*
|
|
* 1. Redistributions of source code must retain the above copyright notice,
|
|
* this list of conditions and the following disclaimer.
|
|
*
|
|
* 2. Redistributions in binary form must reproduce the above copyright
|
|
* notice, this list of conditions and the following disclaimer in the
|
|
* documentation and/or other materials provided with the distribution.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
|
|
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
|
|
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
|
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
|
|
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
|
|
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
|
|
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
|
|
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
|
|
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
|
|
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
|
|
* POSSIBILITY OF SUCH DAMAGE.
|
|
*
|
|
*/
|
|
|
|
#include <errno.h>
|
|
#include "basethread.h"
|
|
|
|
|
|
namespace Winix
|
|
{
|
|
|
|
|
|
BaseThread::BaseThread() : thread_signal(PTHREAD_COND_INITIALIZER)
|
|
{
|
|
synchro = 0;
|
|
thread_id = 0;
|
|
work_mode = 0;
|
|
wake_up_was_called = false;
|
|
}
|
|
|
|
|
|
BaseThread::~BaseThread()
|
|
{
|
|
}
|
|
|
|
|
|
void BaseThread::set_main_log_buffer(PT::WTextStream * log_buffer)
|
|
{
|
|
main_log.SetLogBuffer(log_buffer);
|
|
}
|
|
|
|
|
|
void BaseThread::set_main_file_log(PT::FileLog * file_log)
|
|
{
|
|
main_log.SetFileLog(file_log);
|
|
}
|
|
|
|
|
|
void BaseThread::Mode(int mode)
|
|
{
|
|
work_mode = mode;
|
|
}
|
|
|
|
|
|
pthread_t BaseThread::ThreadId()
|
|
{
|
|
return thread_id;
|
|
}
|
|
|
|
|
|
|
|
bool BaseThread::Lock()
|
|
{
|
|
return synchro->Lock();
|
|
}
|
|
|
|
|
|
void BaseThread::Unlock()
|
|
{
|
|
synchro->Unlock();
|
|
}
|
|
|
|
|
|
|
|
bool BaseThread::IsExitSignal()
|
|
{
|
|
bool res = true;
|
|
|
|
if( Lock() )
|
|
{
|
|
res = synchro->was_stop_signal;
|
|
Unlock();
|
|
}
|
|
|
|
return res;
|
|
}
|
|
|
|
|
|
bool BaseThread::BaseInit()
|
|
{
|
|
bool init_status = false;
|
|
|
|
if( Lock() )
|
|
{
|
|
init_status = Init(); // your virtual method
|
|
Unlock();
|
|
}
|
|
|
|
return init_status;
|
|
}
|
|
|
|
|
|
void BaseThread::BaseUninit()
|
|
{
|
|
if( Lock() )
|
|
{
|
|
Uninit(); // your virtual method
|
|
Unlock();
|
|
}
|
|
}
|
|
|
|
|
|
|
|
|
|
bool BaseThread::BaseSignalReceived()
|
|
{
|
|
bool make_do = false;
|
|
|
|
wake_up_was_called = false;
|
|
|
|
try
|
|
{
|
|
make_do = SignalReceived(); // your short-time virtual method (objects are locked)
|
|
}
|
|
catch(...)
|
|
{
|
|
}
|
|
|
|
return make_do;
|
|
}
|
|
|
|
|
|
// this is called only if your SignalReceived() returned true
|
|
void BaseThread::BaseDo()
|
|
{
|
|
try
|
|
{
|
|
Do(); // your long-time virtual method (objects are *not* locked)
|
|
}
|
|
catch(...)
|
|
{
|
|
}
|
|
}
|
|
|
|
|
|
// use it with Lock and Unlock
|
|
bool BaseThread::WaitForSignal()
|
|
{
|
|
if( synchro->was_stop_signal || wake_up_was_called )
|
|
return true;
|
|
|
|
return pthread_cond_wait(&thread_signal, &synchro->mutex) == 0;
|
|
}
|
|
|
|
|
|
|
|
// you should use this method with: synchro->Lock() and Unlock()
|
|
void BaseThread::WakeUpThread()
|
|
{
|
|
wake_up_was_called = true;
|
|
pthread_cond_signal(&thread_signal);
|
|
}
|
|
|
|
|
|
// use it with Lock and Unlock
|
|
// it breaks only if there was a stop signal or the time has expired
|
|
bool BaseThread::WaitForSignalSleep(time_t second)
|
|
{
|
|
timespec t;
|
|
int res;
|
|
|
|
if( synchro->was_stop_signal )
|
|
return true;
|
|
|
|
t.tv_sec = time(0) + second;
|
|
t.tv_nsec = 0;
|
|
|
|
do
|
|
{
|
|
res = pthread_cond_timedwait(&thread_signal, &synchro->mutex, &t);
|
|
}
|
|
while( res == 0 && !synchro->was_stop_signal );
|
|
// above condition means there was a signal
|
|
// but it was not a stop signal so we should still wait
|
|
|
|
return res == 0 || res == ETIMEDOUT;
|
|
}
|
|
|
|
|
|
void BaseThread::WaitForThread()
|
|
{
|
|
pthread_join(thread_id, 0);
|
|
}
|
|
|
|
|
|
void BaseThread::SignalLoop()
|
|
{
|
|
bool make_do;
|
|
|
|
do
|
|
{
|
|
if( Lock() )
|
|
{
|
|
make_do = false;
|
|
|
|
if( WaitForSignal() ) // automatically unlock, wait and lock again when signal comes
|
|
if( !synchro->was_stop_signal )
|
|
make_do = BaseSignalReceived(); // your short-time virtual method will be called (objects locked)
|
|
|
|
Unlock(); // unlocking from WaitForSignal()
|
|
|
|
if( make_do )
|
|
BaseDo(); // your long-time virtual method will be called (objects *not* locked)
|
|
}
|
|
}
|
|
while( !IsExitSignal() );
|
|
}
|
|
|
|
|
|
void * BaseThread::StartRoutine(void * this_object)
|
|
{
|
|
BaseThread * base = reinterpret_cast<BaseThread*>(this_object);
|
|
|
|
if( base->synchro )
|
|
{
|
|
if( base->BaseInit() )
|
|
{
|
|
if( base->work_mode == 0 )
|
|
base->SignalLoop();
|
|
else
|
|
base->Work();
|
|
|
|
base->BaseUninit();
|
|
}
|
|
}
|
|
|
|
base->save_log();
|
|
pthread_exit(0);
|
|
return 0;
|
|
}
|
|
|
|
|
|
|
|
bool BaseThread::StartThread()
|
|
{
|
|
pthread_attr_t attr;
|
|
|
|
pthread_attr_init(&attr);
|
|
pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_JOINABLE);
|
|
|
|
int res = pthread_create(&thread_id, &attr, StartRoutine, this);
|
|
|
|
pthread_attr_destroy(&attr);
|
|
|
|
return res == 0;
|
|
}
|
|
|
|
|
|
} // namespace Winix
|
|
|
|
|