Question

我很久以前就习惯用C编码而且我似乎丢失了我的印章。

我遇到了一些动态char *数组的奇怪行为。我有一个curses窗口（在另一个文件中实现），用于在我的本地计算机上显示活动CU和TTY设备的字符串列表。我尝试使用char **对calloc数组进行零初始化，然后每次（dir->d_name）匹配我的字符串比较时将其增大一个适当的大小。

问题是这些字符串在运行程序时显示一半的时间，而另一半时间显示为乱码或空白。

我怀疑这是与malloc()或calloc()相关的内存/指针问题，但我几天都无法确定我的误解。这是代码：

#include <stdio.h>
#include "funcs.h"
#include <ncurses.h>
#include <dirent.h>
#include <string.h>
#include <stdlib.h>

#define arraysize(ar) sizeof(ar)/sizeof(ar[0])

int main(void) 
{
    char ch;
    const char tty[4] = "tty.";
    const char cu[3] = "cu.";
    const char *directory = "/dev/";
    const char **devices = calloc(0,sizeof(const char *));

    // Initial device list gen //

    DIR *d;
    struct dirent *dir; 
    int count=0;
    d = opendir(directory); 
    if (d != NULL)
    {
        while((dir = readdir(d)))
        {
            if(strncmp(dir->d_name,tty,4)==0 || strncmp(dir->d_name,cu,3)==0)
            {
                count++;
                devices = realloc(devices,(count+1)*sizeof(const char *));
                devices[count-1] = (dir->d_name);
            }   
        }
        closedir(d);    
    }
printf("\n%d\n",count);

struct display MAIN = screen_init();

// Display List //

list_devices(devices,count,MAIN.devpad);
while(ch != 'q')
{
    ch = getch();
}
endwin();
}

感谢您的时间/精力。

Answer 1

这里最大的问题是你没有复制dir d_name缓冲区。正如手册所说：

readdir()返回的数据可能会被后续调用同一目录流的readdir()覆盖。

这意味着你需要明确地复制它：

devices[count-1] = strdup(dir->d_name);

这应该消除很多乱七八糟的东西。

C中动态char **的奇怪行为

1 个答案: