5

我有一个带有 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 *'

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

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

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

有没有更好的办法 ？

谢谢

4

4 回答 4

5

这是安全的。该错误与将 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 个字节怎么办？

于 2011-08-10T10:41:17.927 回答
1

非 const * 的所有内容都可以在 C 中安全地转换为 const *。它是保存的。

于 2011-08-10T10:22:21.437 回答
1

这是安全的。警告（我认为）弹出是因为您正在从未签名转换为已签名。

于 2011-08-10T10:25:07.207 回答
0

它的安全，char < uint8_t 的范围。

于 2011-08-10T10:29:54.273 回答