C# / VB.NET - Aiuto su struttura C++ da inviare tramite socket

Forum - C# / VB.NET - Aiuto su struttura C++ da inviare tramite socket

Thejuster (Admin)
Guru^2

Messaggi: 2305
Iscritto: 04/05/2008

Postato alle 23:06
Lunedì, 05/11/2018

Salve ragazzi ho un problema che non riesco a risolvere in nessun modo.
Spero che potete aiutarmi a trovare una soluzione.

Dunque ho la necessità di inviare una struttura tramite pacchetto TCP ad un server C++.

Il problema e che:
O mi crasha il programma , o il server non intercetta correttamente i valori ricevuti.

Parto dal principio.

Struttura C++

Codice sorgente - presumibilmente C++

struct login_session_data {
        uint32 account_id;                      ///also GID
        long login_id1;
        long login_id2;
        char sex;                       /// 'F','M','S'
 
        char userid[NAME_LENGTH];       /// account name
        char passwd[PASSWD_LENGTH]; // 23+1 for plaintext, 32+1 for md5-ed passwords
        int passwdenc;                  /// was the passwd transmited encrypted or clear ?
        char md5key[20];                /// md5 key of session (each connection could be encrypted with a md5 key)
        uint16 md5keylen;               /// len of the md5 key
 
        char lastlogin[24];             ///date when last logged, Y-M-D HH:MM:SS
        uint8 group_id;                 ///groupid of account
        uint8 clienttype;               /// ???
 
        uint8 client_hash[16];          ///hash of client
        int has_client_hash;            ///client ha sent an hash
 
        int fd;                         ///socket of client
};

Funzione dove il server riceve il socket e ne ricava i valori

Codice sorgente - presumibilmente Delphi

/**
 * Received a connection request.
 * @param fd: file descriptor to parse from (client)
 * @param sd: client session
 * @param command: packet type sent
 * @param ip: ipv4 address (client)
 *  S 0064 <version>.L <username>.24B <password>.24B <clienttype>.B
 *  S 0277 <version>.L <username>.24B <password>.24B <clienttype>.B <ip address>.16B <adapter address>.13B
 *  S 02b0 <version>.L <username>.24B <password>.24B <clienttype>.B <ip address>.16B <adapter address>.13B <g_isGravityID>.B
 *  S 01dd <version>.L <username>.24B <password hash>.16B <clienttype>.B
 *  S 01fa <version>.L <username>.24B <password hash>.16B <clienttype>.B <?>.B(index of the connection in the clientinfo file (+10 if the command-line contains "pc"))
 *  S 027c <version>.L <username>.24B <password hash>.16B <clienttype>.B <?>.13B(junk)
 *  S 0825 <packetsize>.W <version>.L <clienttype>.B <userid>.24B <password>.27B <mac>.17B <ip>.15B <token>.(packetsize - 0x5C)B
 * @param fd: fd to parse from (client fd)
 * @return 0 failure, 1 success
 */
static int logclif_parse_reqauth(int fd, struct login_session_data *sd, int command, char* ip){
        size_t packet_len = RFIFOREST(fd);
 
        if( (command == 0x0064 && packet_len < 55)
        ||  (command == 0x0277 && packet_len < 84)
        ||  (command == 0x02b0 && packet_len < 85)
        ||  (command == 0x01dd && packet_len < 47)
        ||  (command == 0x01fa && packet_len < 48)
        ||  (command == 0x027c && packet_len < 60)
        ||  (command == 0x0825 && (packet_len < 4 || packet_len < RFIFOW(fd, 2))) )
                return 0;
        else {
                int result;
                char username[NAME_LENGTH];
                char password[PASSWD_LENGTH];
                unsigned char passhash[16];
                uint8 clienttype;
                bool israwpass = (command==0x0064 || command==0x0277 || command==0x02b0 || command == 0x0825);
 
                // Shinryo: For the time being, just use token as password.
                if(command == 0x0825) {
                        char *accname = RFIFOCP(fd, 9);
                        char *token = RFIFOCP(fd, 0x5C);
                        size_t uAccLen = strlen(accname);
                        size_t uTokenLen = RFIFOREST(fd) - 0x5C;
 
                        if(uAccLen > NAME_LENGTH - 1 || uAccLen == 0 || uTokenLen > NAME_LENGTH - 1  || uTokenLen == 0)
                        {
                                logclif_auth_failed(sd, 3);
                                return 0;
                        }
 
                        safestrncpy(username, accname, uAccLen + 1);
                        safestrncpy(password, token, uTokenLen + 1);
                        clienttype = RFIFOB(fd, 8);
                }
                else
                {
                        safestrncpy(username, RFIFOCP(fd,6), NAME_LENGTH);
                        if( israwpass )
                        {
                                safestrncpy(password, RFIFOCP(fd,30), PASSWD_LENGTH);
                                clienttype = RFIFOB(fd,54);
                        }
                        else
                        {
                                memcpy(passhash, RFIFOP(fd,30), 16);
                                clienttype = RFIFOB(fd,46);
                        }
                }
                RFIFOSKIP(fd,RFIFOREST(fd)); // assume no other packet was sent
 
                sd->clienttype = clienttype;
                safestrncpy(sd->userid, username, NAME_LENGTH);
                if( israwpass )
                {
                        ShowStatus("Request for connection of %s (ip: %s)\n", sd->userid, ip);
                        safestrncpy(sd->passwd, password, NAME_LENGTH);
                        if( login_config.use_md5_passwds )
                                MD5_String(sd->passwd, sd->passwd);
                        sd->passwdenc = 0;
                }
                else
                {
                        ShowStatus("Request for connection (passwdenc mode) of %s (ip: %s)\n", sd->userid, ip);
                        bin2hex(sd->passwd, passhash, 16); // raw binary data here!
                        sd->passwdenc = PASSWORDENC;
                }
 
                if( sd->passwdenc != 0 && login_config.use_md5_passwds )
                {
                        logclif_auth_failed(sd, 3); // send "rejected from server"
                        return 0;
                }
 
                result = login_mmo_auth(sd, false);
 
                if( result == -1 )
                        logclif_auth_ok(sd);
                else
                        logclif_auth_failed(sd, result);
        }
        return 1;
}

ora leggendo i commenti in questa funzione, esattamente questo che vado ad utilizzare:

*  S 0064 <version>.L <username>.24B <password>.24B <clienttype>.B

Interpreto questa riga in questo modo.

0064  è il comando  quindi  essendo come parametro 'int command'  ho fatto  0x64.

<version> è 0   ma .L  non ho capito cos'è..

<username>  ho impostato come char[] username.

da codice

Codice sorgente - presumibilmente C# / VB.NET

char username[NAME_LENGTH];

24B Leggendo il sorgente posso intuire che 24B stia come 24byte di grandezza per il campo.
sia per username che per password.

Clienttype è 0 posso intuire "Forse" che B stia per byte, ma leggendo la struttura login_session_data
clienttype è impostato come "uint8 clienttype;"

quindi ho scritto ushort.

Ecco la mia classe C#.

Codice sorgente - presumibilmente C++

[StructLayout(LayoutKind.Sequential, CharSet = CharSet.Ansi)]
    struct DataPacket
    {
 
        public int command;  //Conterrà 0x64
 
        public long version;   //Conterrà 0
 
        //dichiaro la variabile array char nella forma nativa LPStr 
       //Il server chiede che la sequenza sia di 24byte quindi imposto 
       //la costante per la lunghezza della variabile di 24
        [MarshalAs(UnmanagedType.LPStr, SizeConst = 24)]
        public char[] Username; 
        
        [MarshalAs(UnmanagedType.LPStr, SizeConst = 24)]
        public char[] Password;
 
        //Tipo di client che conterrà 0
        public ushort clienttype;
 
   
        // La chiamata a questo metodo restituirà un array di byte con i contenuti
        // della struttura pronta per essere inviata tramite tcp.
        public byte[] Serialize()
        {
           
           // alloco un array di byte per i dati struct
            var buffer = new byte[Marshal.SizeOf(typeof(DataPacket))];
 
            // Assegna un GCHandle e ottiene il puntatore dell'array
            var gch = GCHandle.Alloc(buffer, GCHandleType.Pinned);
            var pBuffer = gch.AddrOfPinnedObject();
 
            // copia i dati da struct a array e sblocca il puntatore gc
            Marshal.StructureToPtr(this, pBuffer, false);
            gch.Free();
 
            return buffer;
        }
 
 
        // questo metodo deserializza un array di byte nella struct.
        public void Deserialize(ref byte[] data)
        {
            var gch = GCHandle.Alloc(data, GCHandleType.Pinned);
            this = (DataPacket)Marshal.PtrToStructure(gch.AddrOfPinnedObject(), typeof(DataPacket));
            gch.Free();
        }
    }

Dal main

1° Tentativo ( Sicuramente Sbagliato)
credo che l'errore sia qui

public char[] Username;
dichiarare così queste due variabili.

Codice sorgente - presumibilmente C# / VB.NET

//S 0064 <version>.L <username>.24B <password>.24B <clienttype>.B
            DataPacket packet = new DataPacket();
 
            packet.command = 0x64;  Console.WriteLine(packet.command);
            packet.version = 0;
            packet.Username = String.Format("{0}", "Thejuster").ToCharArray();
            packet.Password = String.Format("{0}", "123456789").ToCharArray();
            packet.clienttype = 0;
 
            var bytes = packet.Serialize();
 
            soc.Send(bytes);

Testo quotato

Additional information: Impossibile effettuare il marshalling del tipo DataPacket come struttura non gestita. Non è possibile calcolare una dimensione o un offset significativo.

2° Tentativo modificando le varibili di tipo char[] in string
e cambiando da LPStr in ByValTStr

Anche qui dubito fortemente che inviare una stringa sia la cosa corretta.
Ma questa volta non c'è crash.

codice

Codice sorgente - presumibilmente C# / VB.NET

//S 0064 <version>.L <username>.24B <password>.24B <clienttype>.B
            DataPacket packet = new DataPacket();
 
            packet.command = 0x64;
            Console.WriteLine(packet.command);
            packet.version = 0;
            packet.Username = "Thejuster";
            packet.Password = "123456789";
            packet.clienttype = 0;
 
            var bytes = packet.Serialize();
 
            soc.Send(bytes);

Ecco cosa dice il server

Login tramite mio test:

https://i.postimg.cc/qqypskgp/Cattura.png

Login Tramite client

https://i.postimg.cc/mrCVWJct/Cattura-PNG-5d92eb58c14f593f0 ...

Quando effettuo un login con un client corretto appare

[Status]: Request for connection of "Thejuster" (ip: 127.0.0.1) esempio

mentre qui, e come se il valore della stringa non viene inviato o in un qualche modo si sballa tutto.

Ma ho notato un problema proprio ora.

noto chiaramente che la variabile bytes che contiene tutta la struttura serializzata,
misura 75 mentre guardando il sorgente del server

Codice sorgente - presumibilmente Plain Text

if( (command == 0x0064 && packet_len < 55)

Deve essere < di 55

ecco la screen

https://i.postimg.cc/CLR11gwr/Cattura.png

Dove sbaglio?

Ultima modifica effettuata da Thejuster il 05/11/2018 alle 23:17

https://mire.forumfree.it/ - Mire Engine
C# UI Designer

TheDarkJuster (Member)
Guru^2

Messaggi: 1620
Iscritto: 27/09/2013

Postato alle 2:20
Mercoledì, 07/11/2018

Tu in pratica hai creato una struct in c# che dovrebbe riflettere la struttura del C per poterla serializzare in stile C.

C'è un piccolo problema: il compilatore C ha riordinato i campi e aggiunto del padding tra i campi, e il compilatore c# l'ha fatto in un altro modo, quindi le due strutture sono uguali sulla carta, ma completamente diverse fra loro, probabilmente non hanno niente in comune.

Thejuster (Admin)
Guru^2

Messaggi: 2305
Iscritto: 04/05/2008

Postato alle 9:11
Giovedì, 08/11/2018

mi sembra strano juster.
anche perché la stessa operazione che sto tentando di fare in c#
è stata fatta ed eseguita correttamente in javascript.

https://mire.forumfree.it/ - Mire Engine
C# UI Designer

TheDarkJuster (Member)
Guru^2

Messaggi: 1620
Iscritto: 27/09/2013

Postato alle 10:59
Giovedì, 08/11/2018

Si, è fattibilissimo in ogni linguaggio se conosci gli offset dei campi dati della struttura. Se vuoi puoi farlo anche in cobol o assembly, ma non otterrai niente se non capisci come sono disposti i dati.