С++ Чтение значения строки реестра в char*

При использовании std::string сравнение будет вести себя так, как вы ожидали. Также это исправит еще одну ошибку, когда функция возвращает указатель на локальный буфер.

std::string GetRegistry(const char* StringName)
{
....
return std::string(value);
}

GetRegistry() возвращает char*, поэтому на самом деле вы сравниваете указатели с operator==. Вы должны использовать strcmp() для необработанных сравнений строк char* в стиле C, или лучше использовать надежный класс строк C++, например CString или std::[w]string.

Вот возможная переработка вашей функции с использованием ATL CString:

#include <atlbase.h>
#include <atlstr.h>

CString GetRegistry(LPCTSTR pszValueName)
{
    // Try open registry key
    HKEY hKey = NULL;
    LPCTSTR pszSubkey = _T("SOFTWARE\\Microsoft\\Windows NT\\CurrentVersion\\MCI Extensions");
    if ( RegOpenKey(HKEY_LOCAL_MACHINE, pszSubkey, &hKey) != ERROR_SUCCESS )
    {
        // Error:
        // throw an exception or something...
        //
        // (In production code a custom C++ exception 
        // derived from std::runtime_error could be used)
        AtlThrowLastWin32();
    }

    // Buffer to store string read from registry
    TCHAR szValue[1024];
    DWORD cbValueLength = sizeof(szValue);

    // Query string value
    if ( RegQueryValueEx(
            hKey,
            pszValueName, 
            NULL, 
            NULL, 
            reinterpret_cast<LPBYTE>(&szValue), 
            &cbValueLength) 
         != ERROR_SUCCESS )
    {
        // Error
        // throw an exception or something...
        AtlThrowLastWin32();
    }

    // Create a CString from the value buffer
    return CString(szValue);
}

И тогда вы можете назвать это так:

if ( GetRegistry(_T("First")) == GetRegistry(_T("Second")) )
    ...

Обратите внимание, что этот код будет компилироваться как в сборках ANSI/MBCS, так и в сборках Unicode (он основан на модели Win32 TCHAR).

У вас есть пара проблем с этим исходным кодом.

Во-первых, у вас есть функция с локальной переменной, переменной в стеке, которая возвращает адрес этой переменной, но когда функция возвращается, переменная исчезает, и адрес больше недействителен.

Следующая проблема заключается в том, что вы не сравниваете строки символов. Вместо этого вы сравниваете адрес, возвращаемый функцией, и, если вам повезет, адрес может быть таким же. Поскольку вы вызываете функцию дважды подряд, вам повезло, поэтому адрес один и тот же.

Я предлагаю вам сделать следующее: (1) создать две строки локальных символов в вашей функции, которая вызывает функцию GetRegistry(), и (2) изменить функцию GetRegistry(), чтобы она использовала эти буферы, а не свои собственные. Таким образом, код будет выглядеть примерно так:

char registryEntryOne[1024];
char registryEntryTwo[1024];
DWORD dwRegistryEntryOneLen;
DWORD dwRegistryEntryTwoLen;

registryEntryOne[0] = 0;   // init the registry entry to zero length string
registryEntryTwo[0] = 0;

dwRegistryEntryOneLen = sizeof(registryEntryOne);
GetRegistry ("First", registryEntryOne, &dwRegistryEntryOneLen);
dwRegistryEntryTwoLen = sizeof(registryEntryTwo);
GetRegistry ("Second", registryEntryTwo, &dwRegistryEntryTwoLen);

// two strings are equal if:
//   the lengths are the same
//   at least one of the lengths is non-zero
//   the bytes are the same in the same order
if (dwRegistryEntryOneLen && dwRegistryEntryOneLen == dwRegistryEntryTwoLen && memcmp (registryEntryOne, registryEntryTwo, dwRegistryEntryOneLen) == 0) {
    // strings are equal
} else {
    // strings are not equal
}

Функция GetRegistry() будет выглядеть примерно так:

char* GetRegistry(char* StringName, char *valueBuffer, DWORD *value_length)
{
    DWORD dwType = REG_SZ;
    HKEY hKey = 0;
    const char* subkey = "SOFTWARE\\Microsoft\\Windows NT\\CurrentVersion\\MCI\\Player";

    RegOpenKey(HKEY_LOCAL_MACHINE,subkey,&hKey);
    RegQueryValueEx(hKey, StringName, NULL, &dwType, (LPBYTE)valueBuffer, value_length);
    return  valueBuffer;
}