About custom packets, PLS Help me, Thaks!

[halloway520]

hi every one:

PHP Code:

void CNetProcessIn::OnB983(CMsgStreamBuffer &msg) {

    DEBUG_PRINT_CALL()

    std::n_string recvMsg;
    msg >> recvMsg;
    msg.FlushRemaining();

    char *cstr = new char[recvMsg.length() + 1];
    strcpy(cstr, recvMsg.c_str());

    g_vRank.clear();
    char *tmp = strtok(cstr, "#");
    while (tmp != NULL)
    {
        g_vRank.push_back(tmp);
        tmp = strtok(NULL, "#");
    }

    delete [] cstr;

    if (!g_vRank.empty())
    {
        CIFConvert cc;
        g_Charname->SetText(cc.AnsiToUnicode(g_vRank[0].c_str()));
    }

    reinterpret_cast<void (__thiscall *)(CNetProcessIn *, CMsgStreamBuffer &)>(0x0084CAB0)(this, msg);
} 


The code does what I need, but the sro_client.exe will be crash once in a while. It happens every 3~5 times when i received 0xB983.


I know the problem is at

PHP Code:

reinterpret_cast<void (__thiscall *)(CNetProcessIn *, CMsgStreamBuffer &)>(0x0084CAB0)(this, msg); 


but, i do not know how to get a stable（safe）address,

I can not understand F0 or HB's guide.

I want to know which address I should hook up to and if someone can share that address with me, Thank you very much,

I do not know ASM, last, sorry for my english.：D

[florian0]

The answer is simple: Don't call 0x0084CAB0 in the end. You don't need it.

Explanation:
That function looks like it belongs to the CNetProcessIn implementation I added to SRO_DevKit. B983 is not a standard msgid, therefore I guess it's a custom one.
CNetProcessIn will call your handler function if a message with that Id is received. Your handler has to read all bytes from the message, otherwise the game will think there was an error.

The redirection in the end is only required when a function is hooked. You redirect some function to your own function and call the original function in the end to close the loop.
Since your handler function is entirely your own creation and has no equivalent in the game, you don't need to call any original function in the end (since you don't have one). You can just leave the function without that. It'll work fine.

[halloway520]

Originally Posted by florian0

The answer is simple: Don't call 0x0084CAB0 in the end. You don't need it.

Explanation:
That function looks like it belongs to the CNetProcessIn implementation I added to SRO_DevKit. B983 is not a standard msgid, therefore I guess it's a custom one.
CNetProcessIn will call your handler function if a message with that Id is received. Your handler has to read all bytes from the message, otherwise the game will think there was an error.

The redirection in the end is only required when a function is hooked. You redirect some function to your own function and call the original function in the end to close the loop.
Since your handler function is entirely your own creation and has no equivalent in the game, you don't need to call any original function in the end (since you don't have one). You can just leave the function without that. It'll work fine.

You solved my confusion, The program is working perfectly.

Again, thank you for your great contribution. Thank epvp.

fix.

[devdash]

It's sad when i see things looks shiny but i do not understand it still.
Will i be able to understand what are you guys talking about if i start learning C#?
i started already but i still can't understand anything, maybe on a higher level of C#?

i really wish someday i can understand what are you guys talking about here, because i feel i am not even in the trace race.

give me hints, so i can join the trace <3

[irockalone]

Originally Posted by alydandy

It's sad when i see things looks shiny but i do not understand it still.
Will i be able to understand what are you guys talking about if i start learning C#?
i started already but i still can't understand anything, maybe on a higher level of C#?

i really wish someday i can understand what are you guys talking about here, because i feel i am not even in the trace race.

give me hints, so i can join the trace <3

ahaha, i felt like you're describing me.

[JellyBitz]

Starts with basic knowledge.
is a good start, never is too late.