在将空字符串分配给字符串变量时面临核心转储答案

【问题标题】：facing core dump while assigning the empty string to string variable在将空字符串分配给字符串变量时面临核心转储
【发布时间】：2012-01-30 17:43:32
【问题描述】：

当我的应用程序尝试将文本写入文件时，我面临核心转储。它使用字符串操作，如代码所示。当多个线程尝试调用此函数并尝试写入日志文件时，我的应用程序正在转储核心。我浏览了不同的论坛并尝试搜索字符串分配中的任何问题。我看到我们中的一些人评论了那个字符串赋值语句，因为它不好用。而是使用字符串成员函数。

功能：

void Debug::writeToFile()
{
if(_ptrMutex == NULL)
    _ptrMutex = new Mutex();
_ptrMutex->getLock();
write(_fd,_cacheStr.c_str(),_cacheStr.size());
_cacheStr = ""; //flush the write string
_ptrMutex->releaseLock();
}

cacheWriteString

bool Debug::cacheWriteString(string strIn)
{
if(_ptrMutex == NULL)
    _ptrMutex = new CndMutex();
_ptrMutex->getLock();
char timeStamp[100];

struct timeval  tv;
struct tm   tm;

if (gettimeofday(&tv, NULL)==0 && localtime_r((time_t*)&(tv.tv_sec), &tm)) {
    int ret = strftime(timeStamp,sizeof(timeStamp),"%F-%T",&tm);
    snprintf(timeStamp+ret, sizeof(timeStamp)-ret, ".%03ld", tv.tv_usec/1000);
} else
Time::getTimeStamp(timeStamp,100);
_cacheStr = _cacheStr + "[" + timeStamp + "] " + strIn;
if(_cacheStr.size() >= _maxCacheSize)
{
    _ptrMutex->releaseLock();
    return true;
}
_ptrMutex->releaseLock();
return false;
}

这个函数被称为

void Debug::LOG_PRINT_ERROR(char* ptrFormat,...)
{
va_list argList;
va_start(argList,ptrFormat);

if(_logType == CONSOLE_LOG) //console log
{
    cout<<"#### [ERROR] ";
    vprintf(ptrFormat,argList);
    cout<<endl;
}
else if(_logType == SYS_LOG)// [syslog]
{
    syslog(LOG_ERR,"%s","[ERROR]####");
    vsyslog(LOG_ERR,ptrFormat,argList);
}
else if(_logType == FILE_LOG)
{
    char str[1024];
    //SBYTE4 ret = vsprintf(str,ptrFormat,argList);
    SBYTE4 ret = vsnprintf(str,1023,ptrFormat,argList);
    str[1023]='\0';
    if(ret > -1)
    {
        string dataStr = "[ERROR]####";
        dataStr = dataStr +str+"\n";
        if(cacheWriteString(dataStr))
        {
            if(chkFile() == DIM_PASS)
            {
                writeToFile();
            }
            else
            {
                cout<<"#### LOG_ERROR() Unable to Write Data To File: "<<_fileName <<endl;
            }
        }
    }
    else
    {
        cout<<"#### LOG_ERROR() Output Error Is Encountered: "<<_fileName <<endl;
    }
}
va_end(argList);

}

Core Dump 详情如下：

Thread 1 (Thread 8426):
#0  0x00a2a402 in __kernel_vsyscall ()
#1  0x0072bdf0 in raise () from /lib/libc.so.6
#2  0x0072d701 in abort () from /lib/libc.so.6
#3  0x0545651a in ?? () from /usr/lib/libstdc++.so.6
#4  0x05456552 in std::terminate() () from /usr/lib/libstdc++.so.6
#5  0x0545668a in __cxa_throw () from /usr/lib/libstdc++.so.6
#6  0x053ed1ef in std::__throw_length_error(char const*) () from /usr/lib/libstdc++.so.6
#7  0x0543211d in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::_Rep::_S_create(unsigned int, unsigned int, std::allocator<char> const&) () from /usr/lib/libstdc++.so.6
#8  0x05433e28 in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_mutate(unsigned int, unsigned int, unsigned int) () from /usr/lib/libstdc++.so.6
#9  0x05433fca in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_replace_safe(unsigned int, unsigned int, char const*, unsigned int) () from /usr/lib/libstdc++.so.6
#10 0x05434065 in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::assign(char const*, unsigned int) () from /usr/lib/libstdc++.so.6
#11 0x0815e9a8 in Debug::writeToFile() ()
#12 0x08161866 in Debug::LOG_PRINT_ERROR(char*, ...) ()
#13 0x0812bcc6 in DimInternalMsgHandler::handlePeerStatusIndication(DimPeerStatusInd*) ()
#14 0x0812c52a in DimInternalMsgHandler::handleInternalMessage(unsigned char*, int) ()
#15 0x0812aa05 in DimDanIfController::handleInMessage(NwPacket&) ()
#16 0x081ab28f in WorkerThreadPool::onEvent(int, nxs_util::EventMessage64*) ()
#17 0x0021dd94 in nxs_util::EventHandler64::workerProc(int) () from /home/nextgen/NXS/lib/libnxsutil.so
#18 0x001c193e in nxs_util::WorkerThread::run() () from /home/nextgen/NXS/lib/libnxsutil.so
#19 0x001c5b36 in nxs_util::Thread::__Thread_run(void*) () from /home/nextgen/NXS/lib/libnxsutil.so
#20 0x0087f832 in start_thread () from /lib/libpthread.so.0
#21 0x007d4e0e in clone () from /lib/libc.so.6

我们怀疑我们在代码中使用的 <_cachestr> 语句存在问题。我们正在考虑更改为 _cacheStr.clear();但是我们现在无法复制这个核心。

如果有人对此有任何想法或遇到过，请告诉我们。

非常感谢您的意见。

【问题讨论】：

一个小提示：在创建互斥锁时，writeToFile 中可能存在竞争条件。如果两个线程可以进入函数并且一个在检查之后但在创建之前被抢占，那么两个线程都可以创建互斥锁。

标签： c++

【解决方案1】：

这看起来真的，真的很狡猾：

if(_ptrMutex == NULL)
    _ptrMutex = new Mutex();

如果两个线程进入这个块并动态创建自己的互斥锁怎么办？是否有任何理由不能在初始化时创建单个互斥锁并避免“创建互斥锁”竞争条件？

【讨论】：

这确实很狡猾。如果没有更好的初始化位置，您可以使用pthread_do_once 正确执行此操作。
这在初始化时有效，因为这点在初始化期间有两个线程来时有效。但是这里的应用程序正在连续运行，然后转储核心。由于 Debug 类具有单个实例，因此总是最后一个互斥锁将在线程中使用。我接受此代码不应该存在。但崩溃会在初始化时发生。
@user1177666：很抱歉，如果两个线程在初始化时进入这个函数是不安全的。任何事情都可能发生，例如双重解锁一个互斥体并完全破坏同步对象。这可能会在随后的任何时间点造成破坏性副作用。
非常感谢您的意见。我考虑到这一点。但目前无论发生什么核心转储，我想都不是因为这个问题。你怎么看？
@user1177666：除非您解决此问题，否则您永远无法确定。从表面上看，_cacheStr = "" 不应该失败，除非对象已损坏或您执行一些线程不安全的操作。鉴于此，您确实需要修复您发现的所有潜在线程错误。我没有看到任何具体证据表明此错误不是您崩溃的根本原因。

【解决方案2】：

如果任何其他线程可以访问_cacheStr（即使只是为了检查它），而_ptrMutex 未被持有，这很容易导致这样的崩溃。

【讨论】：

其他线程必须通过这个互斥锁机制，除非线程正在释放它。所以所有的线程都会去正常的锁定和解锁机制。

【解决方案3】：

我看到函数 writeToFile 有锁定，并且 cacheStr 应该是一个全局变量，但我没有看到函数 cacheWriteString，所以我认为这是 cacheStr 变量不受保护的竞争条件函数cacheWriteString。 2个线程可以同时访问/修改字符串。

【讨论】：