Question

我有以下代码，它从函数char*

获取返回值

cDestDrive = ReadFromRegistry(HKEY_CURRENT_USER,NDSPATH,szDestPathRoot);

我能够在cDestDrive内读取值，直到我分配它为止。我分配它的那一刻：

 CString strServerAddress = cDestDrive;

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

修改从注册表中读取的代码

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

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

Answer 2

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

TCHAR tSZValue[MAX_PATH] = {0};

接下来是：

char* cpRegKeyVal=tSZValue;

这很危险，您正在亲身体验最终结果！

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

Answer 3

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

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

Answer 4

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

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

char *值在赋值期间被破坏

4 个答案: