Question

我正在为系统编程类开发一个自定义shell。我们被指示实施内置setenv()和unsetenv()命令，并附带检查putenv() 的手册页。

我的问题是，setenv(char*, char*, int)和putenv(char*)似乎根本不起作用。我执行输入命令的代码如下：

//... skipping past stuff for IO redirection
pid = fork();
if(pid == 0){
    //child
    if(!strcmp(_simpleCommands[0]->_arguments[0],"printenv")){
        //check if command is "printenv"
        extern char **environ;
        int i;
        for(i = 0; environ[i] != NULL; i++){
            printf("%s\n",environ[i]);
        }
        exit(0);
    }
    if(!strcmp(_simpleCommands[0]->_arguments[0],"setenv")){
        //if command is "setenv" get parameters char* A, char* B
        char * p = _simpleCommands[0]->_arguments[1];
        char * s = _simpleCommands[0]->_arguments[2];

        //putenv(char* s) needs to be formatted A=B; A is variable B is value
        char param[strlen(p) + strlen(s) + 1];
        strcat(param,p);
        strcat(param,"=");
        strcat(param,s);
        putenv(param);
        //setenv(p,s,1);
        exit(0);
    }
    if(!strcmp(_simpleCommands[0]->_arguments[0],"unsetenv")){
        //remove environment variable
        unsetenv(_simpleCommands[0]->_arguments[0]);
        exit(0);
    }
    //execute command
    execvp(_simpleCommands[0]->_arguments[0],_simpleCommands->_arguments);
    perror("-myshell");
    _exit(1);
}

//omitting restore IO defaults...

如果我运行printenv它运行正常，但如果我尝试使用putenv()或setenv()设置新变量，则printenv()命令会返回完全相同的内容，所以它似乎没有用。

作为旁注，问题可能不在于函数或我如何调用它们，因为我的shell正在执行命令，就好像它必须格式化通配符（*或?）我不确定应该发生什么。

Answer 1

在检查命令行之前，您似乎无条件地调用fork。但是一些shell内置命令需要在父进程中运行，因此它们的效果仍然存在。所有操纵环境的内置函数都属于这一类。

顺便说一句，如果我正在编写shell，我不会尝试使用C库的环境操作函数。我使用三个参数main，将envp复制到我完全控制下的数据结构中，然后将其反馈回execve。这部分是因为我是一个控制狂，部分是因为它几乎不可能做任何复杂的setenv和/或putenv并且没有内存泄漏。有关血腥的详细信息，请参阅this older SO question。

Answer 2

是什么让你觉得它不起作用？我在下面写了一个简单的测试用例......它按预期工作。

确保在相同过程中调用setevn和prientevn。

#include <stdlib.h>
#include <assert.h>
int main()
{


 char * s= "stack=overflow";

 int ret =  putenv(s);

 assert(ret == 0);

 //printout all the env
 extern char **environ;
        int i;
        for(i = 0; environ[i] != NULL; i++){
            printf("%s\n",environ[i]);
        }


 return 0; 

}
pierr@ubuntu:~/workspace/so/c/env$ ./test | grep stack
stack=overflow

setenv，unsetenv，putenv

2 个答案: