Является ли код ошибки Win32 122 несколько мягким в этом случае?

Question

следующие результаты строки в GetLastError() возврата кода ошибки 122 (= ERROR_INSUFFICIENT_BUFFER)

CString str = CString("'") + _T("%s") + CString("'"); 

Но это происходит только под VS2005 и не бывает в VS2015. Тем не менее я не вижу повреждения памяти или чего-либо в VS2005, а переменная str содержит правильное значение. Должно ли это быть проблемой, связанной с предоставлением кода ошибки?

Причина этого, кажется, происходит из-за стечения широких символов и простых символьных строк и исправление просто sorround как остальные строки с _T("") так строка кода будет выглядеть следующим образом:

CString str = CString(_T("'")) + _T("%s") + CString(_T("'")); 

Но что действительно ли код ошибки 122 означает в исходной строке, когда только одна строка была Unicode? Что случилось на самом деле или это скорее похоже на предупреждение в этом случае?

Answer 1

В VS 2015 вы можете воспроизвести ошибку с CString("a") (если Unicode установлен) или только CStringW("a")

#include <iostream> 
#include <atlstr.h> 

int main() 
{ 
    CStringW("a"); 
    DWORD err = GetLastError(); 
    std::cout << err << "\n"; //<= error 122, ERROR_INSUFFICIENT_BUFFER 
    return 0; 
} 

Это происходит потому, что CString использует WinAPI MultiByteToWideChar для преобразования ANSI "a" в Unicode L"a". Отладка через исходный код в "atlmfc\include\cstringt.h", мы видим, что в какой-то момент он вызывает следующую функцию:

static int __cdecl GetBaseTypeLength(_In_z_ LPCSTR pszSrc) throw() 
{ 
    // Returns required buffer size in wchar_ts 
    return ::MultiByteToWideChar(_AtlGetConversionACP(), 0, pszSrc, -1, NULL, 0)-1; 
} 

По какой-то причине есть -1 в конце. Я не знаю, почему это возможно, возможно, это необходимо для других функций CString, но в этом случае он вызывает ошибку ERROR_INSUFFICIENT_BUFFER при следующем вызове MultiByteToWideChar. Преобразование может быть примерно упрощено до следующего:

int main() 
{ 
    int nDestLength = MultiByteToWideChar(CP_ACP, 0, "a", -1, NULL, 0) - 1; 
    wchar_t *pszDest = new wchar_t[32]; 

    //ERROR_INSUFFICIENT_BUFFER occurs here because nDestLength is short by 1: 
    MultiByteToWideChar(CP_ACP, 0, "a", -1, pszDest, nDestLength); 
    DWORD err = GetLastError(); 
    std::cout << err << "\n"; 
    return 0; 
} 

nDestLength слишком мал, потому что он не учитывает нулевой терминатор. CString сортирует это позже, но ошибка остается. Это хорошая причина не обращать внимание на GetLastError, если функция не работает.

Как вы заметили, эту ошибку можно избежать, используя макрос _T, потому что CString больше не понадобится MultiByteToWideChar. Или еще лучше, использовать префикс L или CString::Format

CString str = CString(L"'") + L"%s" + CString(L"'"); 

Answer 2

GetLastError() имеет смысл только после того, как какой-то системный вызов вернул ошибку. Так как у вашего кода нет системного вызова, GetLastError() может что-то вернуть.

Возможно, значение, которое вы видите, является последней ошибкой последнего отказа системного вызова. Или, может быть, это некоторая ошибка, которая произошла из класса CString, но она обрабатывается там.

TL; DR; Здесь нет ошибки.