Повреждение кучи обнаружено после вызова удаления в Visual C ++?

Question

Повреждение кучи обнаружено после вызова удаления в Visual C ++?

0

Я пытаюсь запустить свой c++ код, введение которого в этот новый код привело к повреждению памяти, может ли кто-нибудь помочь мне в этом, что бы это вызвало. После вызова delete я столкнулся с этой проблемой. Я также попытался разместить newArgs[SZ] = '\0'; после каждого вызова strncpy.

Ошибка говорит:

КОПИРОВАНИЕ HEAP ОБНАРУЖЕНА после нормального блока (# 274) в 0X00C09600 и т.д.

int main(HINSTANCE hInstance, HINSTANCE hPrevInstance, char* argc, int nShowCmd){

    MyClass *obj;
    char args[] = " hello world";
    int SZ = strlen(args);
    int argsLength = 0;

    if(argc != NULL)
        argsLength=strlen(argc);
    SZ+=argsLength;
    char *newArgs = new char[SZ];
    strncpy(newArgs, "",SZ);

    if(argc != NULL)
        strncpy(newArgs, argc,argsLength);

    StrCat(newArgs,args);

    obj = new MyClass(newArgs);
    delete[] newArgs;

    return 0; 
}

Juseeth 14 нояб. 2014, в 04:20

Источник

1

Какой SIZE ? Он используется, но никогда не объявляется.
Igor Tandetnik 14 нояб. 2014, в 03:02
0

Вы добавляете argsLength в SIZE не в SZ , поэтому newArgs не будет достаточно большим.
The Dark 14 нояб. 2014, в 03:02
1

int main(HINSTANCE hInstance, HINSTANCE hPrevInstance, char* argc, int nShowCmd) Что это?
drescherjm 14 нояб. 2014, в 03:05
1

{std::string newArgs = args; newArgs += argc; } Это, вероятно, то, что весь этот код сводится к.
PaulMcKenzie 14 нояб. 2014, в 03:08
0

Вероятно, он хотел написать SZ вместо SIZE, но проблема в том, что SZ должен быть еще одним для завершающего нулевого символа.
ElefEnt 14 нояб. 2014, в 03:09
0

@drescherjm Скорее всего, WinMain переименован в main помощью ключа /ENTRY , по непонятным причинам.
Igor Tandetnik 14 нояб. 2014, в 03:31

Показать ещё 4 комментария

Теги:

c++

visual-c++

visual-studio-2010

heap-memory

heap-corruption

2 ответа

Ещё вопросы

Какой SIZE ? Он используется, но никогда не объявляется.
Вы добавляете argsLength в SIZE не в SZ , поэтому newArgs не будет достаточно большим.
int main(HINSTANCE hInstance, HINSTANCE hPrevInstance, char* argc, int nShowCmd) Что это?
{std::string newArgs = args; newArgs += argc; } Это, вероятно, то, что весь этот код сводится к.
Вероятно, он хотел написать SZ вместо SIZE, но проблема в том, что SZ должен быть еще одним для завершающего нулевого символа.
@drescherjm Скорее всего, WinMain переименован в main помощью ключа /ENTRY , по непонятным причинам.

Werner Henze · Answer 1 · 2014-11-14T07-25-00.000Z

Вы извлекаете длину строки argc. Это без конечного нулевого байта:

argsLength=strlen(argc);

Позже вы копируете argc в свой буфер. Но вы копируете только символы argsLength, которые являются строкой без конечного нулевого байта. strncpy будет только копировать содержимое строки, а не добавлять конечный нулевой байт (см. man strncpy).

strncpy(newArgs, argc,argsLength);

Сразу же после strncpy вы добавляете другую строку в newArgs.

StrCat(newArgs,args);

В зависимости от содержимого newArgs после выделения (который может быть случайным, при отладке он, вероятно, будет заполнен специальным шаблоном) strcat fill не находит конечный нулевой байт в newArgs и, таким образом, будет читаться за пределами выделенного буфера (пока он не будет находит нулевой байт) и добавит ваши строковые аргументы там - где-то в куче и вне выделенной памяти. Это куча коррупции.

Кроме:

При распределении памяти вам понадобится еще один байт для конечного нулевого байта.
Какое использование strncpy (..., "", SZ)?
Используйте std :: string, а не подверженные ошибкам строки C.
Вы пропускаете выделенный объект MyClass.
В C++ строковые литералы являются const.

Я бы переписал вашу программу таким образом (не компилировал и не тестировал ее):

int main(HINSTANCE hInstance, HINSTANCE hPrevInstance, char* argc, int nShowCmd)
{
    const char args[] = " hello world";
    std::string newArgs;
    if(argc != NULL)
        newArgs = argc;
    newArgs += args;

    // Use the line that you prefer
    MyClass obj1(newArgs.c_str());
    std::unique_ptr<MyClass> obj2(new MyClass(newArgs.c_str()));

    return 0; 
}

Как я могу получить длину argc и использовать это strncpy (..., "", SZ) для инициализации массива с пустыми символами изначально, так как он заполняет мусор по умолчанию.
длина аргумента argc: символы strlen (argc) плюс один завершающий нулевой байт. Таким образом, при объединении str1 и str2 целевой буфер должен иметь длину strlen (str1) + strlen (str2) +1 байт. strncpy (..., "", SZ) не будет копировать ни один байт (если SZ == 0), или только завершающий нулевой байт "" (если SZ> 0). Если вы хотите заполнить массив нулевыми байтами (в этом нет необходимости, потому что вы перезаписываете их позже), вам нужно использовать memset или std :: fill.
но вы сказали, что argc не имеет нулевого завершающего символа, поэтому strlen определяет его длину?
@suj: Нет, я этого не говорил. Если argc - это «me», то у вас есть три байта: «m», «e» и завершающий нулевой байт «\ 0», поэтому память, на которую указывает argc, имеет длину 3 байта. strlen (argc) возвращает 2, потому что в строке два символа. Вы действительно должны Google / узнать о том, как строки представлены в C :-).

Jonathan Mee · Answer 2 · 2014-11-14T01-19-00.000Z

Я не могу видеть вашу реализацию MyClass но я готов поспорить, что она хранит копию указателя, переданного в конструктор, и что она также пытается работать с этим после вашего вызова на delete.

Я настоятельно рекомендую вам остановить new char* в C++ и использовать strings.