VPF::malloc возвращает NULL - Форум программистов

malor

Дата 5.4.2012, 09:53 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 632
Регистрация: 17.11.2006

Репутация: нет
Всего: нет

Код

void GetPathSession()
{
    int SizeAppData;
    wchar_t* AppData;

    wchar_t* pathEvent;
    wchar_t* pathDir;

    AppData = _wgetenv(_T("APPDATA"));
    SizeAppData = wcslen(AppData);

    pathEvent = (wchar_t *)malloc(SizeAppData + 1);
    wcscpy(pathEvent, AppData);
    wcscat(pathEvent, _T("\\Winamp\\winamp.ini"));
    
    pathDir = (wchar_t *)malloc(SizeAppData + 1);

    dwError = GetLastError();
    wcscpy(pathDir, AppData);
    wcscat(pathDir, _T("\\Winamp\\"));
}

Win7 64-bit / RAM 4 Gb / MSVS2008

feodorv

Дата 5.4.2012, 10:03 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

А чему равно SizeAppData?
Судя по всему, правильное значение этой переменной перезаписывется в момент wcscpy или wcscat, так как первый заказ памяти идёт побайтно (а нужно по-символьно: sizeof(wchar_t) * (SizeAppData + 1)), плюс, всё-таки, как-то заказывать надо с избытком, расчитывая на wcscat(pathEvent, _T("\\Winamp\\winamp.ini")), а на _T("\\Winamp\\winamp.ini") размер заказанной памяти не расчитан...

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

malor

Дата 5.4.2012, 11:23 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 632
Регистрация: 17.11.2006

Репутация: нет
Всего: нет

Спасибо, feodorv !

Код

void GetPathSession()
{
    int SizeAppData;
    wchar_t* AppData;

    AppData = _wgetenv(_T("APPDATA"));
    SizeAppData = wcslen(AppData);

    wchar_t* pathEvent;
    wchar_t* pathDir;
    pathEvent = (wchar_t*)malloc(sizeof(wchar_t)*(SizeAppData + 1) + 36);
    wcscpy(pathEvent, AppData);
    wcscat(pathEvent, _T("\\Winamp\\winamp.ini"));                // 18 символов
    
    pathDir = (wchar_t*)malloc(sizeof(wchar_t)*(SizeAppData + 1) + 16);

    dwError = GetLastError();
    wcscpy(pathDir, AppData);
    wcscat(pathDir, _T("\\Winamp\\"));                            // 8 символов
}

feodorv

Дата 5.4.2012, 12:14 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

Цитата(malor @ 5.4.2012, 12:23

)

pathEvent = (wchar_t*)malloc(sizeof(wchar_t)*(SizeAppData + 1) + 36);
wcscpy(pathEvent, AppData);
wcscat(pathEvent, _T("\\Winamp\\winamp.ini")); // 18 символов

Как мне кажется, логичнее сделать так:

Код


pathEvent = (wchar_t*)malloc(sizeof(wchar_t)*(SizeAppData + 1 + 18));
    wcscpy(pathEvent, AppData);
    wcscat(pathEvent, _T("\\Winamp\\winamp.ini"));                // 18 символов

или даже через #define и sizeof...

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »