[英]Data corruption in malloc'ed memory
每当我尝试访问使用malloc
获取的malloc
中的数据时,数据就会损坏
我正在编写一个程序,该程序读取Linux目录并在“字符串数组”(c中的char **数组)中写入文件和子目录的名称。 它使用dirent.h
功能(如readdir()
。 readdir返回一个Dirent结构,该结构具有dname[256]
,即目标目录中文件/子目录的名称。 我将Dirent字符串(char *)等同于char **数组中已分配位置的索引
我基本上有一个walk_path()
函数,该函数读取目录条目并将其名称写入到已分配的位置,然后返回该位置
data_t* walk_path(char* path) {
int size = 0;
if(path == NULL){
printf("NULL path\n");
return NULL;
}
struct dirent* entry;
DIR* dir_l = opendir(path);
if(dir_l == NULL) {
char** data = (char**)malloc(sizeof(char*) * 2);
data[0] = path;
data_t* ret = (data_t*)malloc(sizeof(data_t));
ret->data = data;
ret->size = 1;
return ret;
}
while((entry = readdir(dir_l)) != NULL) {
if(!strcmp(entry->d_name, ".") || !strcmp(entry->d_name, ".."))
continue;
size++;
}
closedir(dir_l);
char** data = (char**)malloc(sizeof(char*) * size + 1);
int loop_v = 0;
dir_l = opendir(path);
while((entry = readdir(dir_l)) != NULL && loop_v < size) {
if(!strcmp(entry->d_name, ".") || !strcmp(entry->d_name, ".."))
continue;
data[loop_v] = entry->d_name;
loop_v++;
}
closedir(dir_l);
data_t* ret = (data_t*)malloc(sizeof(data_t*));
ret->size = (size_t)size;
ret->data = data;
return ret;
}
以及合并路径功能,该功能可以获取两个目录并将其数据写入单个数组
char** merge_path(char* path, char* path2) {
data_t* path_data = walk_path(path);
data_t* path2_data = walk_path(path2);
if(path_data == NULL || path2_data == NULL) {
printf("Merge failed, couldn't get path data\n");
return NULL;
}
char** new_dir_info = (char**)malloc(sizeof(char*) * (path2_data->size + path_data->size) );
if(new_dir_info == NULL)
return NULL;
int loop = 0;
while(loop < path_data->size) {
new_dir_info[loop] = path_data->data[loop];
loop++;
}
loop = 0;
while(loop < path2_data->size) {
new_dir_info[loop + path_data->size] = path2_data->data[loop];
loop++;
}
free(path_data);
free(path2_data);
return new_dir_info;
}
合并路径函数返回的char**
数组始终具有损坏的数据,即字符数组中的字符已损坏,而不是指针本身,尽管我希望它具有从它具有的目录条目传递给它的字符串随机字符串。
我单步执行代码,发现合并路径功能中的数据已损坏,错误源仍可能源自walk_path()
。
这个
data_t* ret = (data_t*)malloc(sizeof(data_t*));
必定是
data_t* ret = (data_t*)malloc(sizeof(data_t));
通常,在C中,不需要对void
指针进行强制转换,因此可以删除代码中对malloc
所有强制转换,这使上面的行类似于:
data_t* ret = malloc(sizeof(data_t*));
更多在排除盲蝽象这一个更好的阶梯从类型加倍到远malloc
吃调用内部malloc()
但最好使用可变分配与对其操作,这样一起:
data_t* ret = malloc(sizeof *ret);
也是这条线
data[loop_v] = entry->d_name;
复制指向条目名称的指针,而不是名称本身。
考虑使用
data[loop_v] = strdup(entry->d_name);
它动态地为entry->d_name
指向的副本分配空间。
替代代替
char**data;
限定
char (*data)[sizeof entry->d_name]; /* Array of pointers to char[as many char as entry->d_name is defined to have] */
要么
char (*data)[sizeof ((struct dirent*)NULL)->d_name]; /* Array of pointers to char[as many char as entry->d_name is defined to have] */
并像这样分配给它(遵循上述建议的模式):
data = malloc((size /* + 1 */) * sizeof *data); /* Not sure what the idea behind this +1 is. */
而不是
data[loop_v] = strdup(entry->d_name);
做
strcpy(data[loop_v], entry->d_name);
如果采用这种方法,则需要相应地调整data_t.data
的定义。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.