下面删除文件夹的程序会不会造成内存泄露？

[martenyong]

BOOL DeleteDirectory(CString psDirName)
{
int res=PathFileExists(psDirName);

//如果文件夹存在，则删除
if(res!=0)
{
int nLength = strlen(psDirName);
char *NewPath = new char[nLength+2];
strcpy(NewPath,psDirName);
NewPath[nLength] = '\0';
NewPath[nLength+1] = '\0';

SHFILEOPSTRUCT FileOp;
ZeroMemory((void*)&FileOp,sizeof(SHFILEOPSTRUCT));

FileOp.fFlags = FOF_NOCONFIRMATION;
FileOp.hNameMappings = NULL;
FileOp.hwnd = NULL;
FileOp.lpszProgressTitle = NULL;
FileOp.pFrom = NewPath;
FileOp.pTo = NULL;
FileOp.wFunc = FO_DELETE;

return SHFileOperation(&FileOp) == 0;
}
else
{
return 1；
}

功能上是能实现文件夹的删除的，但是我想问一下这样删除的话会不会造成内存泄露，
应该怎么改进，谢谢！

[jwybobo2007]

char *NewPath = new char[nLength+2];
这个需要释放

[cpp2017]

char *NewPath = new char[nLength+2];

没有delete 当然会泄露。
为什么要用动态呢，用个静态的就ok

char NewPath[MAX_PATH]={0};

[Eleven]

你的char *NewPath = new char[nLength+2];
在哪里delete掉NewPath呢？

[gradual]

return前需要添加
delete [] NewPath;
NewPath = NULL;

[Eleven]

TCHAR NewPath[MAX_PATH] = {0};
写成这样可能会好一些

[muyouniao]

是啊，有道理

[martenyong]

[Quote=引用 5 楼 visualeleven 的回复:]
TCHAR NewPath[MAX_PATH] = {0};
写成这样可能会好一些
[/Quote]

就这样处理了，谢谢大家！