Указатель и удаление в C ++

Question

Указатель и удаление в C ++

0

void *mm(void *pTo,void*pOr,size_t s)
{
    assert((pTo)&&(pOr));
    char* pt = (char*)pTo;
    char* po = (char*)pOr;
    while(s-->0)
        *pt++ = *po++;
    return pTo;
}

int _tmain(int argc, _TCHAR* argv[])
{

    char *p1 = new char;
    char *p2 = "permutation";
    p1 = (char *)mm(p1,p2,4);
    printf(p1);
    //delete p1;
    return 0;
}

Выше мой тестовый проект. Как я могу удалить новую память символов? Я всегда получаю сообщение "HEAP CORRUPTION DETECTED error... CRT обнаружил, что приложение записано в память после окончания кучного буфера". Заранее спасибо.

Gent 17 дек. 2013, в 08:55

Источник

0

Лучше даже не называть новые или удалять. Используйте std :: vector для хранения памяти динамического размера.
Martin York 18 дек. 2013, в 07:01

Теги:

c++

visual-c++

winapi

2 ответа

0

вы не выделяете достаточное количество байтов "в конце" p1. В частности, вы выделяете только один байт, но копируете 4 байта в местоположение. Следовательно, вы пишете 3 байта за выделенное пространство.

Вместо этого выделите p1 как:

char *p1 = new char[4];

S.C. Madsen 17 дек. 2013, в 05:18

Ещё вопросы

Лучше даже не называть новые или удалять. Используйте std :: vector для хранения памяти динамического размера.

parrowdice · Accepted Answer · 2013-12-17T05-21-00.000Z

Вы даете p1 1 новый символ

char *p1 = new char;

но вы пытаетесь написать 4

while(s-->0)
        *pt++ = *po++;

Вы пишете в память, которой у вас нет.

Делать:

char *p1 = new char[4];

Затем удалите:

delete[] p1;

Поскольку результирующий указатель позже интерпретируется как строка (с помощью printf ), необходимо либо выделить 5 символов (еще один для нулевого терминатора), либо скопировать только 3 символа и использовать четвертый для нулевого терминатора. В любом случае, нулевой терминатор не требуется.