new , delete导致的内存溢出问题,求专家帮忙看一下;

hnyztzh 2011-12-07 12:00:40
这段代码到底那内存溢出了?先用new,再用delete;这个理论上没什么问题呀!求专家帮忙看一下;
if (oFile.Open(m_ResultTitle,CFile::modeRead,&oEx)) // 以读文件方式 打开文件
{
const int nLength = (int)oFile.GetLength(); / 获取文件的大小长度
char* pszBuf = new char[nLength+2];
ZeroMemory(pszBuf,nLength+1); // ZeroMemory没有返回值,在pszBuf,中填充nLength+1个字节0;
oFile.Read(pszBuf,nLength);
CString m_File=pszBuf;

if(m_str2.IsEmpty())
{
if (m_File.Find(m_str1) > -1)
{
return TRUE;
}
}
else if (m_File.Find(m_str1) > -1 && m_File.Find(m_str2) > -1)
{
return TRUE;
}

oFile.Close();
delete [] pszBuf;
}

Detected memory leaks!
Dumping objects ->
e:\toolsuits\taskhis\taskhisview.cpp(339) : {2062} normal block at 0x036B6090, 19457 bytes long.
Data: <700001 1 > 37 30 30 30 30 31 09 31 09 B7 C9 D0 C5 B6 CC D0
e:\toolsuits\taskhis\taskhisview.cpp(339) : {2041} normal block at 0x036B1460, 19457 bytes long.
Data: <700001 1 > 37 30 30 30 30 31 09 31 09 B7 C9 D0 C5 B6 CC D0
e:\toolsuits\taskhis\taskhisview.cpp(339) : {2020} normal block at 0x036AC830, 19457 bytes long.
Data: <700001 1 > 37 30 30 30 30 31 09 31 09 B7 C9 D0 C5 B6 CC D0
e:\toolsuits\taskhis\taskhisview.cpp(339) : {1999} normal block at 0x036A7C00, 19457 bytes long.
Data: <700001 1 > 37 30 30 30 30 31 09 31 09 B7 C9 D0 C5 B6 CC D0
...全文
211 10 打赏 收藏 转发到动态 举报
写回复
用AI写文章
10 条回复
切换为时间正序
请发表友善的回复…
发表回复
MagicFuzzX 2011-12-07
  • 打赏
  • 举报
回复
if (m_File.Find(m_str1) > -1)
{
//泄漏
return TRUE;
}
}
else if (m_File.Find(m_str1) > -1 && m_File.Find(m_str2) > -1)
{
//泄漏
return TRUE;
}


建议写成__try__finally
yayafu 2011-12-07
  • 打赏
  • 举报
回复
没delete你就return了,当然要泄露了
sharptiger123 2011-12-07
  • 打赏
  • 举报
回复
每个return都要考虑delete
ouyh12345 2011-12-07
  • 打赏
  • 举报
回复
这是内存泄露的问题
没有在所有出口处释放内存,也没有关闭文件
CppCoder 2011-12-07
  • 打赏
  • 举报
回复
return 前delete
hnyztzh 2011-12-07
  • 打赏
  • 举报
回复
嗯,各位大牛,正解,我没有在所有出口处释放内存,而且需要关闭文件;
学习了,多谢!
hushoubo 2011-12-07
  • 打赏
  • 举报
回复
正解。
[Quote=引用 8 楼 jiuzhoulh 的回复:]
if(flag == 1)
return TRUE;

直接 return flag ;更简洁
[/Quote]
jiuzhoulh 2011-12-07
  • 打赏
  • 举报
回复
if(flag == 1)
return TRUE;

直接 return flag ;更简洁
liru125504 2011-12-07
  • 打赏
  • 举报
回复
if (oFile.Open(m_ResultTitle,CFile::modeRead,&oEx)) // 以读文件方式 打开文件
{

int flag = 0 ;

const int nLength = (int)oFile.GetLength(); / 获取文件的大小长度
char* pszBuf = new char[nLength+2];
ZeroMemory(pszBuf,nLength+1); // ZeroMemory没有返回值,在pszBuf,中填充nLength+1个字节0;
oFile.Read(pszBuf,nLength);
CString m_File=pszBuf;

if(m_str2.IsEmpty())
{
if (m_File.Find(m_str1) > -1)
{
flag = 1 ;
}
}
else if (m_File.Find(m_str1) > -1 && m_File.Find(m_str2) > -1)
{
flag = 1 ;
}

oFile.Close();
delete [] pszBuf;

if(flag == 1)
return TRUE;

}

16,472

社区成员

发帖
与我相关
我的任务
社区描述
VC/MFC相关问题讨论
社区管理员
  • 基础类社区
  • Web++
  • encoderlee
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告

        VC/MFC社区版块或许是CSDN最“古老”的版块了,记忆之中,与CSDN的年龄几乎差不多。随着时间的推移,MFC技术渐渐的偏离了开发主流,若干年之后的今天,当我们面对着微软的这个经典之笔,内心充满着敬意,那些曾经的记忆,可以说代表着二十年前曾经的辉煌……
        向经典致敬,或许是老一代程序员内心里面难以释怀的感受。互联网大行其道的今天,我们期待着MFC技术能够恢复其曾经的辉煌,或许这个期待会永远成为一种“梦想”,或许一切皆有可能……
        我们希望这个版块可以很好的适配Web时代,期待更好的互联网技术能够使得MFC技术框架得以重现活力,……

试试用AI创作助手写篇文章吧