今天我能够写a simple C++ program that granted a user the "Log on as a service" privilege。其中一部分涉及在LPCWSTR
和LSA_UNICODE_STRING
之间进行转换。这样做的代码在这里:
LSA_UNICODE_STRING StringToLsaUnicodeString(LPCWSTR string) {
LSA_UNICODE_STRING lsaString;
DWORD dwLen = 0;
dwLen = wcslen(string);
lsaString.Buffer = (LPWSTR) string;
lsaString.Length = (USHORT)((dwLen) * sizeof(WCHAR));
lsaString.MaximumLength = (USHORT)((dwLen + 1) * sizeof(WCHAR));
return lsaString;
}
当我在此函数中遇到一些小错误时,我对LsaLookupNames2()
的调用失败,代码为87(十六进制0x57)“参数不正确。”我试图在使用std::wstring
的C ++应用程序中进行此调用,但它失败了。我目前的职能如下:
#if defined(_UNICODE)
LSA_UNICODE_STRING toLsaUnicodeString (std::wstring str) {
LSA_UNICODE_STRING lsaWStr;
DWORD len = 0;
LPWSTR cstr = (LPWSTR)str.c_str();
len = wcslen(cstr);
lsaWStr.Buffer = cstr;
lsaWStr.Length = (USHORT)((len) * sizeof(WCHAR));
lsaWStr.MaximumLength = (USHORT)((len + 1) * sizeof(WCHAR));
return lsaWStr;
}
#endif
我做错了什么?
答案 0 :(得分:4)
您可能会遇到wchar_t*
返回str.c_str()
的终身问题。 str.c_str()
将返回指向生命周期由str
控制的基础字符串的指针。由于str
是按值传递的,因此它将在toLsaUnicodeString
函数的末尾被销毁,从而导致返回的LSA_UNICODE_STRING
指向已被解除分配的内存。为了避免这种情况,您需要在toLsaUnicodeString
函数中复制基础字符串,并将副本与返回的LSA_UNICODE_STRING
相关联,如:
LSA_UNICODE_STRING toLsaUnicodeString (const std::wstring& str) {
LSA_UNICODE_STRING lsaWStr;
DWORD len = 0;
len = str.length();
LPWSTR cstr = new WCHAR[len + 1];
memcpy(cstr, str.c_str(), (len + 1) * sizeof(WCHAR));
lsaWStr.Buffer = cstr;
lsaWStr.Length = (USHORT)((len) * sizeof(WCHAR));
lsaWStr.MaximumLength = (USHORT)((len + 1) * sizeof(WCHAR));
return lsaWStr;
}
由于内存现在已在堆上分配,因此您有责任确保将其解除分配。您可以使用以下功能来处理此问题。
void freeLsaUnicodeString(LSA_UNICODE_STRING& str) {
delete [] str.Buffer;
str.Buffer = 0;
str.Length = 0;
str.MaximumLength = 0;
}
更好的方法是使用RAII来管理内存,并保证在变量不再使用时释放内存。有关此方法的详细信息,请参阅Mr_C64的答案。
答案 1 :(得分:3)
我认为在C ++中执行此操作的正确方法是在原始C结构LSA_UNICODE_STRING周围编写 RAII包装器类。
这个类的构造函数重载正确地初始化它,析构函数释放分配的资源(帮助编写异常安全的代码),并且你可以提供一些operator = overload来做正确的深拷贝。
动态分配的WCHAR缓冲区由std :: vector的实例管理,而不是使用显式的new []和delete [],这简化了代码(例如std :: vector的析构函数将自动释放已分配的内存)
这样的事情:
#include <windows.h> // Win32 SDK header
#include <LsaLookup.h> // LSA_UNICODE_STRING
#include <vector> // std::vector
#include <string> // std::wstring
//
// C++ RAII wrapper to LSA_UNICODE_STRING
//
class LsaUnicodeString
{
public:
LsaUnicodeString()
{
SetEmpty();
}
LsaUnicodeString(const LsaUnicodeString & source)
{
CopyFrom(source);
}
explicit LsaUnicodeString(const std::wstring & source)
{
CopyFrom(source);
}
~LsaUnicodeString()
{
// Nothing to do:
// the string buffer is managed by std::vector data member
}
LsaUnicodeString & operator=(const LsaUnicodeString & source)
{
if (&source != this)
{
CopyFrom(source);
}
return *this;
}
LsaUnicodeString & operator=(const std::wstring & source)
{
CopyFrom(source);
return *this;
}
const LSA_UNICODE_STRING & Get() const
{
return m_us;
}
//
// Implementation
//
private:
LSA_UNICODE_STRING m_us; // raw C structure
std::vector<WCHAR> m_buffer; // string content
void SetEmpty()
{
m_buffer.resize(1);
m_buffer[0] = L'\0'; // end-of-string
m_us.Length = 0;
m_us.MaximumLength = sizeof(WCHAR);
m_us.Buffer = &m_buffer[0];
}
void CopyFrom(const std::wstring & source)
{
if ( source.empty() )
{
SetEmpty();
return;
}
const int len = source.length();
m_buffer.resize(len + 1);
::CopyMemory(&m_buffer[0], source.c_str(), (len+1)*sizeof(WCHAR));
m_us.Length = len * sizeof(WCHAR);
m_us.MaximumLength = m_us.Length + sizeof(WCHAR);
m_us.Buffer = &m_buffer[0];
}
void CopyFrom(const LsaUnicodeString & source)
{
if (source.m_us.Length == 0)
{
SetEmpty();
return;
}
m_buffer = source.m_buffer;
m_us.Length = source.m_us.Length;
m_us.MaximumLength = source.m_us.MaximumLength;
m_us.Buffer = &m_buffer[0];
}
};
答案 2 :(得分:1)
您可以使用RtlInitUnicodeString函数初始化unicode字符串。使用UNICODE_STRING后调用RtlFreeUnicodeString。
UNICODE_STRING和LSA_UNICODE_STRING完全相同。