C - 使用open()和fdopen()时关闭文件的正确方法

rob*_*s35 7 c unix file-io memory-leaks

所以我在C中构建一个Unix minishell,并且正在实现输入,输出和错误重定向,并且遇到了文件问题.我在循环中打开我的文件,在那里我找到重定向操作符,并使用open(),它返回一个fd.然后我相应地分配孩子的fd,并调用执行函数.

当我的shell刚出去找程序,并用execvp()执行它们时,我没有太大的问题.唯一的问题是在提示输入下一个命令行之前,我是否需要在文件描述符上调用close().我担心fd泄漏,但不完全理解它是如何工作的.

使用内置命令时出现了真正的问题.我有一个名为"read"的内置命令,它接受一个参数,一个环境变量名称(可能是一个尚不存在的名称).然后读取提示输入值,并将该值赋给变量.这是一个例子:

% read TESTVAR
test value test value test value
% echo  ${TESTVAR}
test value test value test value
Run Code Online (Sandbox Code Playgroud)

好吧,让我说我尝试这样的事情:

% echo here's another test value > f1
% read TESTVAR < f1
% echo  ${TESTVAR}
here's another test value
Run Code Online (Sandbox Code Playgroud)

这很好用,请记住read在父进程内执行,我不用execvp调用read,因为它是内置的.读取使用获取,需要流变量,而不是fd.所以在浏览了irc论坛之后,我被告知要使用fdopen来从文件描述符中获取流.所以在打电话之前,我打电话给:

rdStream = fdopen(inFD, "r");
Run Code Online (Sandbox Code Playgroud)

然后打电话

if(fgets(buffer, envValLen, rdStream) != buffer) 
{
    if(inFD) fclose(rdStream);
    return -1;
}
if(inFD) fclose(rdStream);
Run Code Online (Sandbox Code Playgroud)

正如您所看到的,目前我正在使用fclose()关闭流,除非它等于stdin(为0).这有必要吗?我需要关闭流吗?或者只是文件描述符?或两者?我很困惑我应该关闭它,因为它们都以不同的方式引用同一个文件.目前我还没有关闭fd,但我认为我绝对应该.我希望有人帮助确保我的shell没有泄漏任何文件,因为我希望它能够在单个会话中执行数千个命令而不会泄漏内存.

谢谢,如果你们想让我发布任何代码,请问.

cni*_*tar 7

标准指出:

fclose()函数应在close()与stream指向的流关联的文件描述符上执行等效的a.

所以打电话fclose就够了; 它也将关闭描述符.

  • 另外叫"关闭"是非常糟糕的**.这是一种双重免费的bug.在没有信号处理程序的单线程程序中,你可能不会遇到麻烦,但是如果在`fclose`和`close`之间有任何其他文件被打开的机会,你将意外地关闭一个不相关的文件并且非常混乱你的程序状态.这可能导致危险的文件损坏错误. (5认同)