Как правильно преобразовать шестнадцатеричную строку в байтовый массив в C?

Question

Мне нужно преобразовать строку, содержащую шестнадцатеричные значения в виде символов, в байтовый массив. Хотя на этот вопрос уже был дан первый ответ, я получаю следующую ошибку:

warning: ISO C90 does not support the ‘hh’ gnu_scanf length modifier [-Wformat]

Так как я не люблю предупреждения и упущения hh просто создает другое предупреждение

warning: format ‘%x’ expects argument of type ‘unsigned int *’, but argument 3 has type ‘unsigned char *’ [-Wformat]

мой вопрос: как это сделать правильно? Для завершения я снова публикую пример кода:

#include <stdio.h>

int main(int argc, char **argv)
{
    const char hexstring[] = "deadbeef10203040b00b1e50", *pos = hexstring;
    unsigned char val[12];
    size_t count = 0;

     /* WARNING: no sanitization or error-checking whatsoever */
    for(count = 0; count < sizeof(val)/sizeof(val[0]); count++) {
        sscanf(pos, "%2hhx", &val[count]);
        pos += 2 * sizeof(char);
    }

    printf("0x");
    for(count = 0; count < sizeof(val)/sizeof(val[0]); count++)
        printf("%02x", val[count]);
    printf("\n");

    return(0);
}

Answer 1

Ты можешь использовать strtol() вместо.

Просто замените эту строку:

sscanf(pos, "%2hhx", &val[count]);

с:

char buf[10];
sprintf(buf, "0x%c%c", pos[0], pos[1]);
val[count] = strtol(buf, NULL, 0);

ОБНОВЛЕНИЕ: Вы можете избежать использования sprintf() используя этот фрагмент вместо:

char buf[5] = {"0", "x", pos[0], pos[1], 0};
val[count] = strtol(buf, NULL, 0);

Answer 2

Вы можете переключить компилятор в режим C99 (hh модификатор длины был стандартизирован в C99), или вы можете использовать unsigned int временная переменная:

unsigned int byteval;
if (sscanf(pos, "%2x", &byteval) != 1)
{
    /* format error */
}
val[count] = byteval;

Answer 3

Почему бы не сделать это без использования sscanf, strol и т. Д. Ниже представлен HexToBin, а в качестве бесплатной пчелы - BinToHex. (Обратите внимание, что изначально были возвращены коды ошибок перечисления через систему регистрации ошибок, а не простой возврат -1.)

unsigned char HexChar (char c)
{
    if ('0' <= c && c <= '9') return (unsigned char)(c - '0');
    if ('A' <= c && c <= 'F') return (unsigned char)(c - 'A' + 10);
    if ('a' <= c && c <= 'f') return (unsigned char)(c - 'a' + 10);
    return 0xFF;
}

int HexToBin (const char* s, unsigned char * buff, int length)
{
    int result;
    if (!s || !buff || length <= 0) return -1;

    for (result = 0; *s; ++result)
    {
        unsigned char msn = HexChar(*s++);
        if (msn == 0xFF) return -1;
        unsigned char lsn = HexChar(*s++);
        if (lsn == 0xFF) return -1;
        unsigned char bin = (msn << 4) + lsn;

        if (length-- <= 0) return -1;
        *buff++ = bin;
    }
    return result;
}

void BinToHex (const unsigned char * buff, int length, char * output, int outLength)
{
    char binHex[] = "0123456789ABCDEF";

    if (!output || outLength < 4) return;
    *output = '\0';

    if (!buff || length <= 0 || outLength <= 2 * length)
    {
        memcpy(output, "ERR", 4);
        return;
    }

    for (; length > 0; --length, outLength -= 2)
    {
        unsigned char byte = *buff++;

        *output++ = binHex[(byte >> 4) & 0x0F];
        *output++ = binHex[byte & 0x0F];
    }
    if (outLength-- <= 0) return;
    *output++ = '\0';
}

Answer 4

Используя предложенное mvp изменение, я создал эту функцию, которая включает проверку ошибок (недопустимые символы и неровная длина).

Эта функция преобразует шестнадцатеричную строку - НЕ предшествующую "0x" - с четным числом символов в указанное количество байтов. Он вернет -1, если встретит недопустимый символ, или если шестнадцатеричная строка имеет нечетную длину, и 0 в случае успеха.

//convert hexstring to len bytes of data
//returns 0 on success, -1 on error
//data is a buffer of at least len bytes
//hexstring is upper or lower case hexadecimal, NOT prepended with "0x"
int hex2data(unsigned char *data, const unsigned char *hexstring, unsigned int len)
{
    unsigned const char *pos = hexstring;
    char *endptr;
    size_t count = 0;

    if ((hexstring[0] == '\0') || (strlen(hexstring) % 2)) {
        //hexstring contains no data
        //or hexstring has an odd length
        return -1;
    }

    for(count = 0; count < len; count++) {
        char buf[5] = {'0', 'x', pos[0], pos[1], 0};
        data[count] = strtol(buf, &endptr, 0);
        pos += 2 * sizeof(char);

        if (endptr[0] != '\0') {
            //non-hexadecimal character encountered
            return -1;
        }
    }

    return 0;
}