传递给 C 中的函数时丢失 char**array 的内容

Question

在我最终创建某种 shell 的持续努力中，我有以下结构（最终基于 execvp()。

struct commands {
    char cmdname[30]; // The name of the command
    enum ActionType action; /* char action[30];  what action to take */
};

struct userinput {
    struct commands theaction; //The chosen action
    char cmdentered[100]; // The cmd entered
    char **anyargs; //The tokenised command
    int argcount; //Argument count
};

然后我使用 malloc 初始化 anyargs 并创建一个字符串数组，每个参数一个字符串传递给 execvp。

然后我获取用户输入，将输入转换为存储在 anyargs 中的标记，并检查字符串以找出需要采取的操作类型并将其存储在枚举中。

所有这些方法都是通过将指针作为方法参数传递给结构 userinput 来完成的——效果很好。但是，当我将指向结构的指针传递给嵌套函数时，char** anyargs 变为空。

我希望我添加的代码能够为答案提供解决方案！在另一个观察中 - 当传递给函数内部的函数时，指针的实际值不会改变 - 只有指针的取消引用内容。

我们将不胜感激任何帮助！我试图将代码剥离到我认为导致问题的区域！ 谢谢！

int main() {

    struct commands cmdlist[4]; //Array of structures with all commands in them
    memset(cmdlist, 0, sizeof(cmdlist));

    struct userinput userentry = { { { 0 } } }; //Structure containing input
    userentry.theaction = cmdlist[0]; //Initialize empty command
    userentry.anyargs = calloc(100, sizeof(char));

    runEntry(&userentry, cmdlist); //Pass struct to function 

    free(userentry.anyargs);

    return 0;
}

int runEntry(struct userinput *userentry, struct commands thecmds[]) {
    int retval = 0;
    int childpid = 0;
    int processStatus;
    printf("\n    ... running cmd: \n\n");

    printUserEntry(userentry); //in printUserEntry, 
                               //userentry->anyargs[0] = NULL - why?
}

Answer 1

您已在 anyargs 中分配了价值 100 字节的 char * 元素。不过，您还没有初始化这些指针。 anyargs[0] 恰好包含 NULL 的事实很好，但不能保证。 malloc() 没有初始化分配的空间。

换句话说，当你说：

userentry.anyargs = malloc(100);

你已经创建了：

userentry.anyargs = {
  ???, // uninitialized char * 
  ???, // and another
  ???, // and another
  ...
  ???  // (100 / sizeof(char *)) entries later
};

您可以在循环中将它们显式初始化为 NULL：

for ( i = 0; i < (100 / sizeof(char *)); ++i )
  userentry.anyargs[i] = NULL;

（或使用calloc() 而不是malloc() 以确保一切都归零）。

或者您可以为它们分配一些空间：

for ( i = 0; i < (100 / sizeof(char *)); ++i )
  userentry.anyargs[i] = malloc(50);  // or some other length

或者直接在runEntry()中设置：

userentry.anyargs[0] = "foo";
userentry.anyargs[1] = strdup(something);