Comment puis-je convertir cette Crypto de C# à C

Question

Ceci est le code C# J'utilise:

public void Decrypt(byte[] @in, byte[] @out, int size) 
{ 
    lock (this) 
    { 
     for (ushort i = 0; i < size; i++) 
     { 
      if (_server) 
      { 
       @out[i] = (byte)(@in[i]^0xAB); 
       @out[i] = (byte)((@out[i] << 4) | (@out[i] >> 4)); 
       @out[i] = (byte)(ConquerKeys.Key2[_inCounter >> 8]^@out[i]); 
       @out[i] = (byte)(ConquerKeys.Key1[_inCounter & 0xFF]^@out[i]); 
      } 
      else 
      { 
       @out[i] = (byte)(ConquerKeys.Key1[_inCounter & 0xFF]^@in[i]); 
       @out[i] = (byte)(ConquerKeys.Key2[_inCounter >> 8]^@out[i]); 
       @out[i] = (byte)((@out[i] << 4) | (@out[i] >> 4)); 
       @out[i] = (byte)(@out[i]^0xAB); 
      } 
      _inCounter = (ushort)(_inCounter + 1); 
     } 
    } 
} 

ce qui est la façon dont je me suis converti à travailler en C.

char* decrypt(char* in, int size, int server) 
{ 
    char out[size]; 
    memset(out, 0, size); 
    for (int i = 0; i < size; i++) 
    { 
     if (server == 1) 
     { 
      out[i] = in[i]^0xAB; 
      out[i] = out[i] << 4 | out[i] >> 4; 
      out[i] = Key2[incounter >> 8]^out[i]; 
      out[i] = Key1[incounter & 0xFF]^in[i]; 
     } 
     else if (server == 0) 
     { 
      out[i] = Key1[incounter & 0xFF]^in[i]; 
      out[i] = Key2[incounter >> 8]^out[i]; 
      out[i] = out[i] << 4 | out[i] >> 4; 
      out[i] = out[i]^0xAB; 
     } 
     incounter++; 
    } 
    return out; 
} 

Cependant, pour certains raison que le C ne fonctionne pas.

Link for the full C# file

Link for the full C file

Link for the C implementation

Answer 1

Une erreur de traduction s'est produite.

C# ligne:

@out[i] = (byte)(ConquerKeys.Key1[_inCounter & 0xFF]^@out[i]); 

Est devenu:

out[i] = Key1[incounter & 0xFF]^in[i]; 

La valeur à droite de la XOR (^) est du mauvais tableau.

En outre, vous renvoyez une variable allouée par pile, ce qui provoque toutes sortes de problèmes.

Change:

char out[size]; 
memset(out, 0, size); 

à:

char *out = (char*)calloc(size, sizeof(char)); 

Answer 2

L'erreur la plus flagrante que je vois est que vous retournez un pointeur vers un tableau de pile alloué, qui va se piétiné par le prochain appel de fonction après decrypt() renvoie. Vous devez malloc() tampon ou passer dans un pointeur vers un tampon inscriptible.

Answer 3

Vous renvoyez une référence à une variable locale qui est illégale. Laissez l'appelant passer dans un tableau ou utilisez malloc() pour créer un tableau dans la méthode.

Answer 4

Je suggère également tourner char en unsigned char car il est plus facile à transporter. Si votre plate-forme suppose que char est la même chose que signed char, l'arithmétique (changements de bits, etc.) ne fonctionnera pas correctement.
Donc, il suffit de spécifier unsigned char explicitement (utilisez un typedef ou incluez <stdint.h> si unsigned char semble trop long pour vous).