Cheat Engine

[NoMercy]

I'm creating a function to add items into colums, this is what I've

[Slugsnack]

I don't actually program C++ but I guess you could use string::c_str ? It would not be LPWSTR, it would be LPCSTR. You could convert it afterwards with MultiByteToWideChar(). I think you might even be able to convert from std::string straight to LPWSTR with that API.

The method I mention is definitely doable but I am sure there is a better way. Again, I don't program C++..

[NoMercy]

void InsertLVItem(std::string First,std::string Sec,std::string Third)
{
LVITEM lvi;
LPCSTR y =First.c_str();
LPWSTR x;
SecureZeroMemory(x,20);
MultiByteToWideChar(CP_ACP,MB_COMPOSITE,y,20,x,20);

Thanks, but I keep getting x is used without being intiziliated, so I tried SecureZeroMemory and also x = ""'; but no succes.

EDIT: I've fixed it, maybe not very good but it works, if someone know something better post please.

void InsertLVItem(std::string First,std::string Sec,std::string Third)
{
LVITEM lvi;
LPCSTR y =First.c_str();
wchar_t x[20];
//SecureZeroMemory(x,20);
//MultiByteToWideChar(CP_ACP,MB_COMPOSITE,y,20,x,20);
mbstowcs(x,y,20);

[Slugsnack]

LPWSTR = long pointer to widechar string. You are simply declaring a 4/8 byte pointer. Use string::length to determine length of the string and declare an array of widechars appropriately. LPWSTR is essentially a pointer to an array. You have not initialised it with anything. It's even worse that you've used SecureZeroMemory in this case. You are basically forcing a buffer overflow..

In response to your edit, you should never hardcode string lengths in these sorts of situations. Either pass it in as a parameter or call the string's length() method. What you have done is a dangerous and bad practice.

[atom0s]

Use std::wstring for wide-char variation of the string library.