FIX: Using CString::operator+= May Cause an Access ViolationLast reviewed: September 19, 1997Article ID: Q142385 |
4.00
WINDOWS NT
kbprg kbbuglist kbfixlist
The information in this article applies to:
SYMPTOMSAn access violation may result after using CString::operator+= where the string on the right hand side of the operator is an empty string. If the debug CRT libraries are installed, an Assertion Failure in Dbgheap.c line 1017 will occur. The "Sample Code" section in this article gives an example that demonstrates this problem.
CAUSEThe CString implementation has been optimized in Visual C++ 4.0 to use reference counting in order to minimize duplication in allocation of memory. When operator+= is used to append an empty string, the code in CString::ConcatInPlace() incorrectly decrements the reference count. If more than one CString is pointing at this data, the call to delete in the CString destructor causes an access violation.
RESOLUTIONTo work around this problem, avoid using the += operator with an empty string of the form:
str1 += _T("");Check for empty CStrings prior to appending:
if (!str2.IsEmpty()) str1 += str2; STATUSMicrosoft has confirmed this to be a bug in the Microsoft products listed at the beginning of this article. This bug was corrected in Visual C++ 4.1.
MORE INFORMATION
Sample Code
/* Compile options needed: default */{ CString str1, str2; str1 = _T("allocspace"); // length of RHS > 0, allocate memory // str1 += str2 will otherwise do a fast copy // if both CStrings are uninitialized str1 = _T(""); // make str1 NULL // operator= will put '\0' at m_pchData[0] str2 = str1; // str1 and str2 share the same m_pchData. // m_pchData now has ref count of 2 str1 += _T(""); // m_pchData has ref count of 1 but both str1 // and str2 use this data. } |
Additional reference words: 4.00 4.10
© 1998 Microsoft Corporation. All rights reserved. Terms of Use. |