Почему возникает предупреждение C6386?

Question

[Павел Каптур]

есть вот такой код, который для каждого объекта из folderList создает свой поток

DWORD hThreadArraySize = sizeof(HANDLE);
	hThreadArraySize*=folderList.size();
	hThreadArray = (HANDLE*)VirtualAlloc(NULL, hThreadArraySize, MEM_COMMIT, PAGE_READWRITE);
        for (std::list<FOLDERDESCRIPTOR>::iterator it = folderList.begin(); it != folderList.end(); it++, i++)
	{
               hThreadArray[i] = (*it).hThread = CreateThread();
	}

И на него выдается предупреждение анализатора кода

Warning C6386 Buffer overrun while writing to 'hThreadArray': the writable size is 'hThreadArraySize' bytes, but '8' bytes might be written.

Почему оно возникает и как исправить? я все размеры проверил, все хорошо. Но что то же ему не нравистя

Answer 1

[AtomKrieg]

LPVOID WINAPI VirtualAlloc(
  _In_opt_ LPVOID lpAddress,
  _In_     SIZE_T dwSize,     //Почему dwSize (hThreadArraySize) у вас тип DWORD, а не SIZE_T
  _In_     DWORD  flAllocationType,
  _In_     DWORD  flProtect
);

Comments

[Павел Каптур]

не надо искать в гугл за меня, это я проверял, оно тут не причем. Проблема в превращении LPVOID в HANDLE*

[AtomKrieg]

Павел Каптур: hThreadArray какой тип?

[Павел Каптур]

AtomKrieg: HANDLE*

[AtomKrieg]

Павел Каптур: Вообще код написан так что могут быть ошибки(я бы в два цикла сделал). Вы устанавливаете i в 0 перед for? Это предупреждение компилятора именно об доступе к массиву по индексу, а не при кастах. В справке МСДН именно такие примеры. Строка об ошибке точно на строке с VirtualAlloc?

[Павел Каптур]

AtomKrieg: индексы обнуляю, я даже скажу больше, если я удаляю присваивание перед виртуал аллок, ошибка проподает

[AtomKrieg]

Павел Каптур: мне кажется что еще folderlist.size на 0 проверить и забить предупреждение прагмой. Это же не ошибка.

[AtomKrieg]

Павел Каптур: вы полный код не приводите. Гадать приходится на кофейной гуще.
Написано 4 минуты назад

[Павел Каптур]

AtomKrieg: тут size вообще ни к чему, тут все уперто в преобразование типа LPVOID -> HANDLE*