en lille hurtig en omkring copy en int ind i en char array

Det ser umiddelbart OK ud.

Hvordan henter du ?

int8 charclass = (int8)row[4];
memcpy(buf_ptr, (char*)&charclass, 1);

kunne laves som:

memcpy(buf_ptr, &row[4], 1);

eller

*buf_ptr = row[4];

men memcpy'en bør virke OK.

ved godt den ser lidt incomplete ud, men det er fordi jeg er ved at samle den, gider ikke skrive alle min int8 udtræk inden jeg har fundet fejlen

int generatenewcharpack(DWORD ptr, const char * name)
{
    CConnection* c = reinterpret_cast <CConnection*> (ptr);
    char            cip[15];
    unsigned int    cp = 0;
    c->PeerInfo (&cip[0], 15, &cp);

    char* query = new char[100];
    sprintf(query,"SELECT * FROM characters WHERE name='%s'", name);

    if(mysql_query(pConn,query))
    {
        // error doing the query
        Log2File(TRUE, "(MySQL) ERROR : %s\n", mysql_error(pConn));
        return 0;
    }
    else
    {
        result = mysql_store_result(pConn);
        row = mysql_fetch_row(result);

        int8 opcode = 0x11;
       
        char buf[32];
        char* buf_ptr= buf; // done so we can mover buf_ptr to were we need to memcpy
       
        memcpy(buf_ptr,(char*)&opcode,1);
        buf_ptr+=1;
       
        char bufname[13];
        char* buf_ptrname= bufname; // done so we can mover buf_ptr to were we need to memcpy
        int8 addbyte = 0x00;
        char * name = (char*)row[2];
        memcpy(buf_ptrname,(char*)name, sizeof(name));
        buf_ptrname+=sizeof(name);
        for (int i = sizeof(name); i < 13; i++)
        {
            if(i < 12)
            {
                memcpy(buf_ptrname,(char*)&addbyte, 1);
                buf_ptrname+=1;
            }
            else
            {
                memcpy(buf_ptrname,(char*)&addbyte, 1);
            }
        }
       
        memcpy(buf_ptr, (char*)bufname, sizeof(bufname));
        buf_ptr+=sizeof(bufname);
        // charname is in, works 100% ok

        // error from here and down.
        int8 charclass = (int8)row[4];
        memcpy(buf_ptr, (char*)&charclass, 1);
        buf_ptr+=1;
        int8 gender = (int8)row[5];
        memcpy(buf_ptr, (char*)&gender, 1);
        buf_ptr+=1;
   


        Log2File(TRUE, "Buffer contains, sizeof buf : %i, sizeof buffname: %i\n", sizeof(buf), sizeof(bufname));
        PacketLog((unsigned char*)&bufname, sizeof(bufname));
        PacketLog((unsigned char*)&buf, sizeof(buf));
        //sendpacket(buf, sizeof(buf), ptr);
        // free mysql result's       
        mysql_free_result(result);
    }
    return 0;
}

btw, den for løkke er fordi jeg SKAL have en charname med 13 bytes, go den storer oxo ok.

og

memcpy(buf_ptr,(char*)&opcode,1);
buf_ptr+=1;

er go nok, den smider 0x11 ind i pakken

og mine database felter ser oxo ok ud

type: int
feltnavn: charclass
har: 1 (i feltet)

char * name = (char*)row[2];
        memcpy(buf_ptrname,(char*)name, sizeof(name));
        buf_ptrname+=sizeof(name);
        for (int i = sizeof(name); i < 13; i++)

er du opmærksom på at sizeof(name) altid er 4 uanset hvorlang name er ??

jeg er 99% sikker på at du mener strlen(name) og ikke sizeof(name)

ja, det opdagede jeg 10 minutter siden, har ikke lige fået det rettet, kan jeg da lige gøre nu

okay rettet det:

eksempel på min log for packetlog buf

11 48 65 6A 73 61 00 00 00 00 00 00 00 00 AC AE
CC CC CC CC CC CC CC CC CC CC CC CC CC CC CC CC

(jeg ved godt de CC er der fordi min buffer er større en de data jeg smider ind i den)

men når jeg er færdig er bufferen 32 bytes stor

Men prøv lige at lav:

        int8 charclass = (int8)row[4];
        memcpy(buf_ptr, (char*)&charclass, 1);
        buf_ptr+=1;
        int8 gender = (int8)row[5];
        memcpy(buf_ptr, (char*)&gender, 1);
        buf_ptr+=1;
   

om til:

        int8 charclass = (int8)row[4];
        memcpy(buf_ptr, (char*)&charclass, 1);
printf("%2X\n",(int)*buf_ptr);
        buf_ptr+=1;
        int8 gender = (int8)row[5];
        memcpy(buf_ptr, (char*)&gender, 1);
printf("%2X\n",(int)*buf_ptr);
        buf_ptr+=1;
   
og se om ikke de er OK.

Jeg tror at enten bliver strenge ødelagt senere eller du henter forkert ud.

jeg bruger win32 api, de der printf er til dos ikk :)

jeg bruger bare min packet logger.

jo

brugte det her

int8 charclass = (int8)row[4];
memcpy(buf_ptr, (char*)&charclass, 1);
Log2File(TRUE, "[0x%2X] charclass", charclass);
buf_ptr+=1;
int8 gender = (int8)row[5];
memcpy(buf_ptr, (char*)&gender, 1);
Log2File(TRUE, "[0x%2X] gender", gender);
buf_ptr+=1;

fik

[16:47:58] [0xAC] charclass
[16:47:58] [0xAE] gender

prøver da lige det sammed med (int8)row[4]

hmm får det samme

hvis jeg logger med (int8)&row[4] får jeg det neden under

[16:53:15] [0x60] charclass
[16:53:15] [0x64] gender

jeg kan da godt se der er noget helt galt der, men fatter ikke hvorfor..

Øh

int8 charclass = (int8)row[4];

(int8)&row[4]

der er da stor forskel på om det er row[4] eller &row[4]

ja, men det er jo ikke det jeg ikke fatter, jeg fatter ikke hvorfor den ikke er 0x01 0x00 :P

Jo men hvis

Log2File(TRUE, "[0x%2X] charclass", charclass);
Log2File(TRUE, "[0x%2X] row[4]", (int8)row[4];

giver det samme så er det jo et lid andet sted vi skal lede efter fejlen.

har tjekket igen, i min database står de rigtige oplysninger

type int charclass : 1
type int gender : 0

henter man dem ud som en int, burde de jo blive 0x01, 0x02, så det må være (skulle jeg mene) i:

1) min query eller
2) et eller andet ligger og rodder der i hukommelsen

hum hvis jeg henter dem ud som en char * får jeg hex valuen af dem

31 (1)
30 (0)

Hm.

Så kan du vel bruge:

int8 charclass = (int8)atoi((char *)row[4]);

int8 gender = (int8)atoi((char *)row[5]);

?

Jeg har iøvrigt lige checket - jeg tror faktisk altid at MySQL returnerer
tekst og ikke binære tal !

dælme hvor du kan... ville ønske jeg var så smart, ville jeg ALDRIG have tænkt på.

smider du ikke et svar ? :)

kommer her