несколько вызовов для realloc(), похоже, вызывают кучевое повреждение

Question

В чем проблема с этим кодом? Он падает каждый раз.

Однократно это неудавшееся утверждение «_ASSERTE (_CrtIsValidHeapPointer (pUserData));», в других случаях это всего лишь ошибка «кучка corppuption».

Изменение размера буфера влияет на эту проблему каким-то странным образом - иногда она падает на «realloc», а иногда и на «free».

Я отлаживал этот код много раз, и нет ничего ненормального в отношении указателей.

char buf[2000]; 
char *data = (char*)malloc(sizeof(buf)); 
unsigned int size = sizeof(buf); 

for (unsigned int i = 0; i < 5; ++i) 
{ 
char *ptr = data + size; 
size += sizeof(buf); 
char *tmp = (char*)realloc(data, size); 
if (!tmp) 
{ 
    std::cout << "Oh no.."; 
    break; 
} 
data = tmp; 
memcpy(ptr, buf, sizeof(buf)); 
} 

free(data); 

Спасибо!

Answer 1

Вы разрушаете кучу. realloc может свободно выбирать, чтобы вернуть вам память из совершенно другого места по мере ее перераспределения, и это аннулирует ваш ptr. Установите ptr после перераспределения.

Answer 2

На второй итерации цикла здесь являются значения

• data указывает на буфер размером sizeof(buf)
• size имеет значение sizeof(buf)

Учитывая эти значения от значения ptr заключается в том, что он указывает на конец буфера, выделенного в data. Это память, не принадлежащая процессу, и операция memcpy пишет об этом и повреждает память.

Answer 3

char *ptr = data + size; 
char *tmp = (char*)realloc(data, size); 
memcpy(ptr, buf, sizeof(buf)); 

Вызов realloc() здесь потенциально может освободить старый буфер, прежде чем вернуться в новую.