我有一个带有 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);
警告消失了,但我想知道这个解决方案是否安全。
有没有更好的办法 ?
Thanks
这是安全的。该错误与将 8 位无符号整数与字符混合有关,如果您仅使用这些字符,则这些字符是有符号的char
.
然而,我看到该函数接受uint8_t
并且确实char
角色算术,所以它应该接受char
s (or const char
s,就此而言)。注意一个字符常量'c'
属于类型char
,并且您在内部表达式中混合了有符号和无符号ihex_decode
,因此您必须小心避免溢出或将负数视为大正数。
最后的风格注释。作为in
未修改,应读取const uint8_t* in
(or const char* in
,如上所述)在参数中。另一个风格错误(可能会导致非常严重的错误)是您接受len
as size_t
,但声明i
循环变量为uint8_t
。如果字符串长度超过 255 个字节怎么办?
本文内容由网友自发贡献,版权归原作者所有,本站不承担相应法律责任。如您发现有涉嫌抄袭侵权的内容,请联系:hwhale#tublm.com(使用前将#替换为@)