CodeGuard и размер выделенного блока

@ijonhson · Регистрация: 09.03.2011

Author24 — интернет-сервис помощи студентам

Доброго времени суток у меня выскакивает ошибка,вернее не ошибка всё работает как надо,CodeGuard ругается на утечку памяти несмотря на то что везде прописано delete в общем код:

C++

void __fastcall TForm1::BitBtn1Click(TObject *Sender)
{
if(SaveDialog1->Execute())
 {
    AnsiString ij,ij1,ij2,ij3,ij4;
    ij=StringReplace(SaveDialog1->FileName,"\\","\\\\",TReplaceFlags()<< rfReplaceAll << rfIgnoreCase)+"\\res.ij";
    char* filepath=new char[ij.Length()];
    strcpy(filepath,ij.c_str());//тут
    ofstream file;
    file.open(filepath,std::ios::out|std::ios::binary);
    file.write((char*)&mas,sizeof(mas));
    file.close();
 CreateDirectory(SaveDialog1->FileName.c_str(),NULL);
 ij1=StringReplace(SaveDialog1->FileName,"\\","\\\\",TReplaceFlags()<< rfReplaceAll << rfIgnoreCase)+"\\rb1.ij";
 ij2=StringReplace(SaveDialog1->FileName,"\\","\\\\",TReplaceFlags()<< rfReplaceAll << rfIgnoreCase)+"\\rb2.ij";
 ij3=StringReplace(SaveDialog1->FileName,"\\","\\\\",TReplaceFlags()<< rfReplaceAll << rfIgnoreCase)+"\\rb3.ij";
 ij4=StringReplace(SaveDialog1->FileName,"\\","\\\\",TReplaceFlags()<< rfReplaceAll << rfIgnoreCase)+"\\rb4.ij";
 Label1->Caption=ij1;
 char *ijj1 = new char[ij1.Length()];
 strcpy(ijj1,ij1.c_str());//ещё тут
 savebit(Image1,ijj1);
 char *ijj2 = new char[ij2.Length()];
 strcpy(ijj2,ij2.c_str()); // и тут
 savebit(Image2,ijj2);
 char *ijj3 = new char[ij3.Length()];
 strcpy(ijj3,ij3.c_str()); //везде на strcpy 
 savebit(Image3,ijj3);
 char *ijj4 = new char[ij4.Length()];
 strcpy(ijj4,ij4.c_str());
 savebit(Image4,ijj4);
 delete[] filepath; 
delete[] ijj1;
 delete[] ijj2;
 delete[] ijj3;
 delete[] ijj4;
 }
}

глючит CodeGuard или я как обычно что-то упустил?
prntscrn:

@volvo · 20.03.2013, 15:57

ijonhson, Length() возвращает длину строки без нуль-терминатора, а strcpy пытается копировать всё, включая '\0', вот и не хватает места...

Сообщение от ijonhson

CodeGuard ругается на утечку памяти

Не на утечку он ругается, а на то, что есть попытка обратиться к 19-му байту, в то время, как размер выделенного блока - всего 18.

@ijonhson · 20.03.2013, 16:16 **[ТС]**

и как быть

C++

1	char *ijj1 = new char[ij1.Length()+1];?

@volvo · 20.03.2013, 16:21

Да, выделять память, достаточную для хранения строки вместе с завершающим символом.

@ijonhson · 20.03.2013, 16:29 **[ТС]**

красота спасибо

@volvo Супер-модератор 32992 / 21298 / 8180 Регистрация: 22.10.2011 Сообщений: 36,593 Записей в блоге: 8
	20.03.2013, 15:57	2
	ijonhson, Length() возвращает длину строки без нуль-терминатора, а strcpy пытается копировать всё, включая '\0', вот и не хватает места... Сообщение от ijonhson CodeGuard ругается на утечку памяти Не на утечку он ругается, а на то, что есть попытка обратиться к 19-му байту, в то время, как размер выделенного блока - всего 18. 1

@volvo Супер-модератор 32992 / 21298 / 8180 Регистрация: 22.10.2011 Сообщений: 36,593 Записей в блоге: 8
	20.03.2013, 16:21	4
	Да, выделять память, достаточную для хранения строки вместе с завершающим символом. 0

@ijonhson 47 / 47 / 15 Регистрация: 09.03.2011 Сообщений: 584
	20.03.2013, 16:29 [ТС]	5
	красота спасибо 0