gcc (GCC) 4.7.2
valgrind-3.8.1
c89
你好,
==1160== Invalid read of size 1
==1160== at 0x8048C94: get_input_values (parse_cmd_input.c:278)
==1160== by 0x8048BA0: parse_input (parse_cmd_input.c:245)
==1160== by 0x80489A1: main (parse_cmd_input.c:50)
==1160== Address 0x40ef02c is 0 bytes after a block of size 4 alloc'd
==1160== at 0x40072C5: calloc (vg_replace_malloc.c:593)
==1160== by 0x8048B28: parse_input (parse_cmd_input.c:239)
==1160== by 0x80489A1: main (parse_cmd_input.c:50)
所以它说地址正在读取分配大小为 4 的零字节,并试图从中读取 1 个字节。但是,我没有越过数组的边界,我正在访问元素 0。
我已经用gdb检查过,元素零包含一个字符。
我的程序没有崩溃,而且似乎工作正常。但它可能会在生产服务器上引起问题。
我不确定我在这里是否正确: 这应该是:
cpy_input = calloc(strlen(input) + 1, sizeof(char*));
或:
cpy_input = calloc(strlen(input) + 1, sizeof(char));
在我的系统上,一个 char 是 1 个字节,指向 char 的指针是 4 个字节。
传入的字符串应该是这样的 "25 b"
int parse_input(const char *input)
{
char *cpy_input = NULL;
int has_error = -1;
if(strlen(input) == 0) {
LOG_ERR("FAILED: Empty string");
return -1;
}
cpy_input = calloc(strlen(input) + 1, sizeof(char));
apr_cpystrn(cpy_input, input, sizeof(cpy_input));
LOG_INFO("[ %s ]", cpy_input);
memset(&channel, 0, sizeof channel);
has_error = get_input_values(cpy_input, &channel);
free(cpy_input);
return has_error;
}
int get_input_values(const char *str, channel_t *channel)
{
size_t i = 0;
size_t k = 0;
int upper_flag = 0;
/* Indicates no digits or command found*/
channel->lower = -1;
channel->upper = -1;
channel->cmd = -1;
#define DIG_BUFFER_SIZE 32
char dig_buffer_lower[DIG_BUFFER_SIZE];
char dig_buffer_upper[DIG_BUFFER_SIZE];
if(strlen(str) == 0) {
LOG_ERR("FAILED: Empty string");
return -1;
}
memset(dig_buffer_lower, 0, DIG_BUFFER_SIZE);
memset(dig_buffer_upper, 0, DIG_BUFFER_SIZE);
LOG_INFO("SIZE %d %d", sizeof(char), sizeof(char*));
/* Increament and check for digits */
for(i = 0; i < DIG_BUFFER_SIZE; i++) {
switch(str[i]) {
case 32: /* ignore space */
continue;
case 45: /* '-' Start upper bounds */
LOG_DEBUG("Found a '-' check upper value");
/* Having a second '-' means the string is invalid */
if(!upper_flag) {
upper_flag = 1;
k = 0;
}
break;
} /* switch */
/* Insert on digits into the lower and upper values */
if(isdigit(str[i])) {
if(upper_flag) {
dig_buffer_upper[k++] = str[i];
LOG_DEBUG("dig_buffer_upper[%d] %s", i, dig_buffer_upper);
}
else {
/* Add to digit buffer */
dig_buffer_lower[k++] = str[i];
LOG_DEBUG("dig_buffer_lower[%d] %s", i, dig_buffer_lower);
}
}
} /* for loop */
非常感谢您的任何建议,
最佳答案
sizeof(cpy_input)
只是 sizeof(char *)
,而不是字符串长度。相反,说:
apr_cpystrn(cpy_input, input, strlen(input) + 1);
或者更好的是,使用裸strcpy
或等价物。此外,也无需使用 calloc
将数组归零,因为无论如何您都将要覆盖它。由于 sizeof(char)
根据定义为 1
,您可以使用以下方法分配数组:
cpy_input = malloc(strlen(input) + 1);
(想一想字符串:您已经在合理的地方使用空终止符,否则 strlen
将崩溃或返回一个巨大的值。一旦您相信strlen
的结果,您保证分配足够的内存给 strcpy
字符串和空终止符。或者,您可以使用 memcpy
以获得可能更高效的副本,因为您知道大小。)
关于c - 在 valgrind 下运行时读取无效,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/17464410/