Reading but not getting correct player position

[Buttars0070]

Is there any particular reason this is returning {1, 0, 0} no matter what?

DWORD m_dwLocalPlayer = 0xA30504;
DWORD m_vecOrigin = 0x134;

float Pos[3];
Pos[0] = RPM<float>(ClientDLL + m_dwLocalPlayer + m_vecOrigin);
Pos[1] = RPM<float>(ClientDLL + m_dwLocalPlayer + m_vecOrigin + 0x4);
Pos[2] = RPM<float>(ClientDLL + m_dwLocalPlayer + m_vecOrigin + 0x8);

[Yemiez]

float Pos[3];
unsigned int localBase = RPM<unsigned int>( ClientDLL + m_dwLocalPlayer );
Pos[0] = RPM<float>(localBase + m_vecOrigin);
Pos[1] = RPM<float>(localBase + m_vecOrigin + 0x4);
Pos[2] = RPM<float>(localBase + m_vecOrigin + 0x8);

I haven't done anything related to CS:GO in a very long time, but the above should probably work.

You should however probably just make a struct that holds 3 floats, that way you only need to call RPM twice (localBase and Pos).

[Buttars0070]

float Pos[3];
unsigned int localBase = RPM<unsigned int>( ClientDLL + m_dwLocalPlayer );
Pos[0] = RPM<float>(localBase + m_vecOrigin);
Pos[1] = RPM<float>(localBase + m_vecOrigin + 0x4);
Pos[2] = RPM<float>(localBase + m_vecOrigin + 0x8);

I haven't done anything related to CS:GO in a very long time, but the above should probably work.

You should however probably just make a struct that holds 3 floats, that way you only need to call RPM twice (localBase and Pos).

Thanks for the help, ill test it out now.

The reason I don't use a Vector is because I'm putting it into a function that wants a float*, although the change would be very easy, this isn't my final version of my private. Thanks though

God knows why that worked, Thanks @Yamiez

/Solved, Close

[Yemiez]

The reason I don't use a Vector is because I'm putting it into a function that wants a float*, although the change would be very easy, this isn't my final version of my private. Thanks though

You could still use a Vector class and just cast the pointer to a float*, the sizes are the same and indexing would still work fine. ^_^

You could also just read 3 floats by creating a special RPM function:

template<typename T, unsigned size>
bool RPM( unsigned int addr, T (&array)[size] )
{
     return ReadProcessMemory( handle_, 
                               reinterpret_cast<void*>( addr ),
                               reinterpret_cast<void*>( &array ),
                               sizeof(T) * size,
                               nullptr ); 
}

Where you could just do this then:

float Pos[3];
RPM( localBase + m_vecOrigin, Pos ); // T and size are deduced from argument inputs

[rwby]

// Solved & Closed