碰到一个微软的bug：CWinAppEx::GetString

在调试公司项目代码的时候，有一个系统设置的功能，里面需要从注册表中去读取数据，因为使用了MFC框架所以，为了简化代码直接使用了CWinAppEx::GetString 、CWinAppEx::SetString、CWinAppEx::GetInt、CWinAppEx::SetInt等等框架内函数，因为使用它之间只需要SetRegistryKey(_T("Application Name"));设置一下就好了，远比自己调用win32 API或者CRegKey类用起来方便多了。

发现一个GetString的在实现的时候有一个bug，起初是我在读取一个注册表String时发现，CWinAppEx::GetString即使传入了第二个参数lpzDefault没能成功访问注册表的话返回还是空字符串！

debug进入代码中看：

首先可以发现GetString实际上是调用的GetSectionString函数：

CString CWinAppEx::GetString(LPCTSTR lpszEntry, LPCTSTR lpszDefault /*= ""*/)

{

    return GetSectionString(_T(""), lpszEntry, lpszDefault);

}

afxwinappex.cpp：

CString CWinAppEx::GetSectionString( LPCTSTR lpszSubSection, LPCTSTR lpszEntry, LPCTSTR lpszDefault /*= ""*/)

{

    ENSURE(lpszSubSection != NULL);

    ENSURE(lpszEntry != NULL);

    ENSURE(lpszDefault != NULL);

    CString strRet = lpszDefault;

    CString strSection = GetRegSectionPath(lpszSubSection);

    CSettingsStoreSP regSP;

    CSettingsStore& reg = regSP.Create(FALSE, TRUE);

    if (reg.Open(strSection))

    {

        reg.Read(lpszEntry, strRet);

    }

    return strRet;

}

从代码中来看，前面几行都没有问题，按F11进入到reg.Read(lpszEntry, strRet);

afxsettingsstore.cpp：

BOOL CSettingsStore::Read(LPCTSTR lpszValueName, CString& strValue)

{

    ENSURE(lpszValueName != NULL);

    strValue.Empty();

    DWORD dwCount = ;

    if (m_reg.QueryStringValue(lpszValueName, NULL, &dwCount) != ERROR_SUCCESS)

    {

        return FALSE;

    }

    if (dwCount == )

    {

        return TRUE;

    }

    LPTSTR szValue = new TCHAR [dwCount + ];

    BOOL bRes = m_reg.QueryStringValue(lpszValueName, szValue, &dwCount) == ERROR_SUCCESS;

    if (bRes)

    {

        strValue = szValue;

    }

    delete [] szValue;

    return bRes;

}

在这里可以看到strValue.Empty();在最开始的时候就被调用了，明显不对，就算你是否有正确的值传进来，也不该首先就直接将default value清空啊，坑啊！

搜索看到国外有个哥们也遇到相同的问题：http://www.bcgsoft.com/cgi-bin/forum/topic.asp?TOPIC_ID=4485

巴特西

碰到一个微软的bug：CWinAppEx::GetString

最新文章

热门文章