ACE_DEBUG的溢出错误

 ACE_DEBUG的溢出错误

这个bug又是inmore发现，错误定位ACE_DEBUG在字符串处理中，可能会引发abort。这个bug在5.6.X，和5.4.X（不能确认），5.7.0的版本都存在，我提交了相应的bug后，也许会在5.7.1版本修正。

我们可以用一个简单的程序验证这个BUG。

#include <conio.h>

#include <iostream>

#include <ace/Log_Msg.h>

 

int main(int, char **)

{

    char out_string[64*1024+1];

    memset(out_string,'@',sizeof(out_string));

    out_string[ACE_MAXLOGMSGLEN] = '/0';

    ACE_DEBUG((LM_ALERT,"%s out_string: wo ai beijing tian an

men.",out_string));

    return 0;

};

其就会引发abort函数，如果从MSVC看堆栈信息如下：

msvcr71d.dll!_NMSG_WRITE(int rterrnum=10)  Line 195  C

msvcr71d.dll!abort()  Line 44 + 0x7     C

ACEd.dll!ACE_OS::abort()  Line 39 + 0x8 C++

ACEd.dll!ACE_Log_Msg::log(const char * format_str=0x00416037,

ACE_Log_Priority log_priority=LM_ALERT, char * argp=0x0011fc78)  Line 2085    

  C++

ACEd.dll!ACE_Log_Msg::log(ACE_Log_Priority log_priority=LM_ALERT, const

char * format_str=0x00416034, ...)  Line 953 + 0x14     C++

TestLog.exe!ace_main_i(int __formal=1, int __formal=1)  Line 18 + 0x77

C++

... ...

稍微浏览了一下，我认为BUG在ACE_Log_Msg::log这个地方，ACE为了输出日志信息，使用了一个长度为ACE_MAXLOGMSGLEN+1的buf作为输出的缓冲区，ACE_MAXLOGMSGLEN的默认长度是4096。但是输出处理过程中，其边界值判断存在一定的问题。其ACE 5.6.1版本的代码如下:

ssize_t

ACE_Log_Msg::log (const ACE_TCHAR *format_str,

                  ACE_Log_Priority log_priority,

                  va_list argp)

{

  ..............

  ssize_t result = 0;

 

  // Check that memory was not corrupted, if it corrupted we can't log anything

  // anymore because all our members could be corrupted.

  if (bp >= (this->msg_ + ACE_MAXLOGMSGLEN+1))

    {

      abort_prog = true;

      ACE_OS::fprintf (stderr,

                       "The following logged message is too long!/n");

    }

其中代码if (bp >= (this->msg_ + ACE_MAXLOGMSGLEN+1))应该修改成为if (bp > (this->msg_ + ACE_MAXLOGMSGLEN+1))，注意应该去掉=符号。如果希望确保日志的缓冲区不再溢出，在这个函数的开始阶段，应该对缓冲区的msg_进行初始化，msg_[MAXLOGMSGLEN] = '/0';

另外在这个函数中的这段代码，效率是太低了。

In fuction ACE_Log_Msg::log ,

 

          if (!skip_nul_locate)

            while (*bp != '/0') // Locate end of bp.

              ++bp;

大家在使用ACE_DEBUG的时候要注意使用的频度和输出信息的大小了。