Endnu et spørgsmål, copy buffer ind i en struct.

Er det ikke bare at assigne ?

struct CS_LoginPacket pckt;
char *username;
char *password;
int ip;
pckt.username = username;
pckt.password = password;
pckt.ip = ip;

nej jeg får den buffer fra en client på mit netværks system

void checkname(char buffer, DWORD ptr)
{
    int packsize = sizeof(buffer);
    struct CS_LoginPacket lp;

    //her mangler
}

jeg får en pakke ind, fjerner min første byte før jeg sender den over (opcode), og skal derefter smide resten ind i min struct.

eg: memcpy(lp, buffer, sizeof(buffer);

ja ved godt den self ikke virker, men bare for at du får ideen

min pakke i bytes f.eks ser sådan her ud

0x32 0x45 0x32 0x45 0x45 0x00 (null) 0x45 0x48 0x63 0x32 0x00 (null) 0x32 0x21 0x11 022

som du kan se er det en string (null) string (null) og sidst dword, det skal så smides ind i min struct for at jeg kan tjekke username og password med det fra databasen.

char *username = buffer;
lp.username = username;
char *password = buffer + strlen(username) + 1;
lp.password = password;
int *ip = (int *)(buffer + strlen(username) + 1 + strlen(password) + 1);
lp.ip = *ip;

Jeg antager at det er:

void checkname(char *buffer, DWORD ptr)

og ikke:

void checkname(char buffer, DWORD ptr)

Og den her kan ikke bruges til noget:

int packsize = sizeof(buffer);

hhmm ser ikke ud som om det virker o.o

smider den ind sådan her:

checkname((char *)&recdata[1], ptr);

starter fra recdata[1] fordi den første byte er opcoden, så der kun er navn & password & ip i pakken.

derefter:

void checkname(char * buffer, DWORD ptr)
{
    struct CS_LoginPacket lp;

    char* username = buffer;
    lp.username = username;
    char* password = buffer + strlen(username) + 1;
    lp.password = password;
    int* ip = (int*)(buffer + strlen(username) + 1 + strlen(password) + 1);
    lp.ip = *ip;

        // test, remove after working.
    Log2File(TRUE, "Login request test %d\n", lp.username);
    PacketLog((unsigned char*)&lp, sizeof(lp));

        // send login ok for test purposes.
    sendloginoktest(ptr);
    return;
}

sorry

Log2File(TRUE, "Login request test %d\n", lp.username);

skulle være

Log2File(TRUE, "Login request test %s\n", lp.username);

men så crasher programmet

aah, loggede lige buffer efter den røg over, der er der kun brugernavnet i, dvs den kutter åbenbart pakken ved først 0x00

whops, den smider det rigtige over, my bad, brugte forkert size til logging lol... :( damn jeg er en fjols..

Det virker ?

nops.

whops min fejl, dword er size 4 pga der er nogen bytes efter der bare skal ignores :( sorry

string username
string password
dword ip[4]

resten ignoreres

#include <windows.h>
#include <iostream>
#include <string>
#include <cstring>

using namespace std;

struct CS_LoginPacket
{
    string username;
    string password;
    DWORD ip;
};

void checkname(char * buffer, DWORD ptr)
{
    struct CS_LoginPacket lp;

    char* username = buffer;
    lp.username = username;
    char* password = buffer + strlen(username) + 1;
    lp.password = password;
    int* ip = (int*)(buffer + strlen(username) + 1 + strlen(password) + 1);
    lp.ip = *ip;

    cout << lp.username << endl;
    cout << lp.password << endl;
    cout << lp.ip << endl;
}

int main()
{
  char b[] = { 0x32, 0x45, 0x32, 0x45, 0x45, 0x00, 0x45, 0x48, 0x63, 0x32, 0x00, 0x32, 0x21, 0x11, 022 };

  checkname(b, 0);

  return 0;
}

udskriver:

2E2EE
EHc2
303112498

hos mig hvilket ser OK ud.

eksempel

    username              password          ip          ignore
[46 64 73 61 66 00] [66 64 73 61 66 00] [3E F3 9E 3C] [00 A8 21]

jeg ved godt det ser UD som om ip er null termineret oxo, men det er den ikke.

#include <windows.h>
#include <iostream>
#include <string>
#include <cstring>

using namespace std;

struct CS_LoginPacket
{
    string username;
    string password;
    DWORD ip;
};

void checkname(char * buffer, DWORD ptr)
{
    struct CS_LoginPacket lp;

    char* username = buffer;
    lp.username = username;
    char* password = buffer + strlen(username) + 1;
    lp.password = password;
    int* ip = (int*)(buffer + strlen(username) + 1 + strlen(password) + 1);
    lp.ip = *ip;

    cout << lp.username << endl;
    cout << lp.password << endl;
    cout << lp.ip << endl;
}

int main()
{
  char b[] = { 0x46, 0x64, 0x73, 0x61, 0x66, 0x00, 0x66, 0x64, 0x73, 0x61, 0x66, 0x00, 0x3E, 0xF3, 0x9E, 0x3C, 0x00, 0xA8, 0x21 };


  checkname(b, 0);

  return 0;
}

giver:

Fdsaf
fdsaf
1017049918

hvilket også ser OK ud for mig.

hum, prøvede at debugge, det ser ud som om min

lp.username indeholder en million milliard af ííííííí (ligner dem lidt).

det er vist hvis jeg bruger den her:

WriteDataStr(hdwnd , "Login request for %s", lp.username);

den skriver bare usernavnet (skulle den gerne om ikke andet) i min editbox.

og den crasher fordi lp.username er større en 256 bytes som jeg har sat til max længde.

lp.username er en string ikke en char*

Prøv:

WriteDataStr(hdwnd , "Login request for %s", lp.username.c_str());

ja, det var der min fejl var, så virker det :P

takker, hvis du smider et svar så får du lige pointsne

svar