0

在下面的代码中，如果我注释掉对“GetCurrentDirectory”的调用，一切正常，但如果我不这样做，那么代码会在它之后中断，不会出现子窗口，但程序不会崩溃。编译器没有给出任何错误。

 char *iniFilePath;
 int lenWritten = GetCurrentDirectory( MAX_PATH, iniFilePath );
 if( lenWritten )
 {
     lstrcat( iniFilePath, iniFileName.c_str() );
     char *buffer;
     GetPrivateProfileString( iniServerSectionName.c_str(), serverIp.c_str(), "", buffer, MAX_PATH, iniFilePath );// server ip
     MessageBox( 0, buffer, 0, 0 );
 }
 else
 {
     MessageBox( 0,0,0,0 );
 }
4

2 回答 2

8

iniFilePath是一个未初始化的指针，GetCurrentDirectory()它试图写入，导致未定义的行为。GetCurrentDirectory()不为调用者分配缓冲区：必须提供。

改成：

char iniFilePath[MAX_PATH]; // or similar.

而不是 using lstrcat()，它在其参考页面上有警告不要使用消息，而是使用 a 构造路径std::string以避免潜在的缓冲区溢出：

const std::string full_file_path(std::string(iniFilePath) + "/" + iniFileName);

请注意与Wimmelbuffer指出的类似问题。

于 2012-12-02T10:36:12.700 回答
0

我会这样做以获取当前目录-

int pathLength = GetCurrentDirectory(0, NULL);
std::vector<char> iniFilePath(pathLength);

GetCurrentDirectory(pathLength, iniFilePath.data());

但是请注意，这不是线程安全的，因为目录可能会在两个调用之间从另一个线程更改，但据我所知，很少有程序会更改当前目录，因此这不太可能成为问题。

于 2012-12-02T10:54:48.757 回答