将 unsigned char * (uint8_t *) 转换为 const char *

Question

我有一个带有 uint8_t * 参数的函数：

uint8_t* ihex_decode(uint8_t *in, size_t len, uint8_t *out)
{
    uint8_t i, hn, ln;

    for (i = 0; i < len; i+=2) {
        hn = in[i] > '9' ? (in[i]|32) - 'a' + 10 : in[i] - '0';
        ln = in[i+1] > '9' ? (in[i+1]|32) - 'a' + 10 : in[i+1] - '0';

        out[i/2] = (hn << 4 ) | ln;
    }

    return out;
}

我将此功能与：

uint8_t data[SPM_PAGESIZE]; // SPM_PAGESIZE = 256 bytes
uint8_t sysex_data[SPM_PAGESIZE/2];
ihex_decode(data, strlen(data), sysex_data);

但在这种情况下，我的编译器（avr-gcc）返回警告：

main.c|89|警告：传递“strlen”参数 1 时的指针目标在符号方面不同 /usr/include/string.h|399|注意：预期为“const char *”，但参数的类型为“uint8_t *”

因此，我通过类型转换 data var 找到了解决方案：

ihex_decode(data, strlen((const char *)data), sysex_data);

警告消失了，但我想知道这个解决方案是否安全。

有没有更好的办法 ？

谢谢

Answer 1

这是安全的。该错误与将 8 位无符号整数与字符混合有关，如果您仅使用char.

然而，我看到该函数接受uint8_t并执行char角色算术，因此它应该接受chars （或const chars，就此而言）。请注意，字符常量'c'的类型为char，并且您在内部表达式中混合有符号和无符号ihex_decode，因此必须小心避免溢出或将负数视为大正数。

最后的风格注释。由于in未修改，因此应在参数中读取const uint8_t* in（或const char* in，如上所述）。另一个样式错误（可能会导致非常严重的错误）是您接受lenas size_t，但将i循环变量声明为uint8_t。如果字符串长度超过 255 个字节怎么办？