char* 值在赋值期间被损坏

Question

我有以下代码，它从函数中获取返回值作为 char*

cDestDrive = ReadFromRegistry(HKEY_CURRENT_USER,NDSPATH,szDestPathRoot);

我能够读取 cDestDrive 内的值，直到我分配它。当我分配它时：

 CString strServerAddress = cDestDrive;

cDestDrive 的值被更改（损坏），并且我无法获取 CString strServerAdres 中的值，任何想法为什么会发生这种情况。

编辑： 从注册表读取的代码

char* CNDSShellExtender::ReadFromRegistry(HKEY hKey,LPCTSTR lpNDS,LPSTR lpRegKey)
{

        HKEY hRegKey=NULL;
        if(hKey==NULL || lpNDS==""||lpNDS==NULL||lpRegKey==""||lpRegKey==NULL)
            MessageBox(NULL,"Reading from Registry Failed!Invalid Path",
                                            _T("Network Drive Solution"),
                                                           MB_ICONERROR);

        LONG lOpenRes=RegOpenKey(hKey,lpNDS,&hRegKey);

        if (lOpenRes!=ERROR_SUCCESS ||lpNDS==NULL) 
            MessageBox ( NULL, "Can not Find Any Server to Connect",
                                            _T("NDSShellExtension"),
                                                     MB_ICONERROR );


        if(lOpenRes==ERROR_SUCCESS && lpNDS!=NULL)
        {
            TCHAR tSZValue[MAX_PATH] = {0};
            DWORD dwBufSize=MAX_PATH;
            LONG lCloseOut;
            LPBYTE lpStorage = reinterpret_cast<LPBYTE>(tSZValue);
            char* cpRegKeyVal=tSZValue;

            if (ERROR_SUCCESS == RegQueryValueEx(hRegKey,lpRegKey , 0, 0, (BYTE*)tSZValue, &dwBufSize))
                {
                    lCloseOut= RegCloseKey(hRegKey);
                    if (lCloseOut != ERROR_SUCCESS) 
                        MessageBox (NULL, "Registry Not Closed", 
                                        _T("NDSShellExtension"),
                                                 MB_ICONERROR );
                    return cpRegKeyVal;
                }
            else
            {
                    lCloseOut= RegCloseKey(hRegKey);
                    if (lCloseOut != ERROR_SUCCESS) 
                    MessageBox (NULL, "Registry Not Closed",
                                    _T("NDSShellExtension"),
                                             MB_ICONERROR );
                    return "";
            }
        }
            return "";
}

Answer 1

我相信您正在返回一个 char* 指向在堆栈上分配的数组，即这一行：

TCHAR tSZValue[MAX_PATH] = {0};

后面跟着：

char* cpRegKeyVal=tSZValue;

这是危险的，您正在经历第一手的最终结果！

编辑：为什么不在函数中直接分配给 CString 并返回它？

Answer 2

该函数返回一个指向 tSZValue 的指针，它是一个局部变量，因此当它超出范围时就不再存在。

Answer 3

您将返回一个指向 tSZValue 的指针，它是一个临时变量，将在函数退出后的某个时间被覆盖。

最简单的解决方案：让 ReadFromRegistry() 返回 CString 而不是 char *。

Answer 4

看起来 ReadFromRegistry 没有分配内存来返回值（或者它确实分配了内存，但它位于堆栈上并在函数返回之前被销毁）。

也许您可以传入对 char * 的引用作为参数，而不是返回 char *，并在 ReadFromRegistry 之外分配内存。