2015-04-28 14 views
5

Znalazłem tę implementację CRC32 w Internecie, trochę ją zmieniłem, ale nie mogę jej uruchomić. Zainicjować go i aktualizować go na każdy bajt dostaję od wejścia, ale mam hash nie jest to, co powinno być ...Implementacja Crc32 C - nie działa

typedef struct { 
    unsigned short xor; 
} xor_context; 
void crc32_init(crc32_context *context) { 
    context->crc = 0xFFFFFFFF; 
} 
void crc32_update(crc32_context *context, unsigned char byte) { 
    uint32_t crc, mask; 

    crc = context->crc; 
    crc = crc^byte; 
    for (int j = 7; j >= 0; j--) { // Do eight times. 
     mask = -(crc & 1); 
     crc = (crc >> 1)^(0xEDB88320 & mask); 
    } 
    context->crc = ~crc; 
} 

Ten jest oryginalny

unsigned int crc32b(unsigned char *message) { 
    int i, j; 
    unsigned int byte, crc, mask; 

    i = 0; 
    crc = 0xFFFFFFFF; 
    while (message[i] != 0) { 
     byte = message[i];   // Get next byte. 
     crc = crc^byte; 
     for (j = 7; j >= 0; j--) { // Do eight times. 
     mask = -(crc & 1); 
     crc = (crc >> 1)^(0xEDB88320 & mask); 
     } 
     i = i + 1; 
    } 
    return ~crc; 
} 
+0

go zmienić wtedy? – Persixty

+0

też nie działa, próbowałem go. – kiro135

+0

Czy jesteś pewien, że wielomian jest poprawny? – fuz

Odpowiedz

2
//typedef struct { 
// unsigned short xor; 
//} xor_context;//--> Not sure what part this plays in the code! 

void crc32_init(crc32_context *context) { 
    context->crc = 0xFFFFFFFF; 
} 

void crc32_update(crc32_context *context, unsigned char byte) { 
    uint32_t crc, mask; 

    crc = context->crc; 
    crc = crc^byte; 
    for (int j = 7; j >= 0; j--) { // Do eight times. 
     mask = -(crc & 1); 
     crc = (crc >> 1)^(0xEDB88320 & mask); 
    } 
    //context->crc = ~crc; //<-- Don't perform for every byte. 
    context->crc = crc; //EDIT: Forgot this! 
} 

//Completes the check. 
uint32_t crc32_complete(crc32_context *context){ 
    return ~context->crc; 
} 
+0

Świetnie, dzięki za pomoc. Byłem zdesperowany przez 3 dni: D – kiro135

+0

@ kiro135 Czy dostałeś moją edycję? Byłem zbyt mocno obciążony usuwaniem "nie". Patrz wyżej. – Persixty

+1

Tak, rozumiem. Dzięki – kiro135