Question

我的问题是关于这个问题：segmentation fault on c K&R fopen and fillbuf。

在这个问题中，在myfopen（）函数中，有一个for循环;我不明白的是，fp将具有从_iob开始的价值，但我不知道fp-＆gt;标志在前三个值（已修复）之后取的值。他们是否会采用零值，还有一个问题是在答案中（参见代码下方）他们说我们必须使用malloc为fp提供内存空间但是fp已经提供了内存空间，因为_iob数组有空格和fp将继续获取数组元素的地址，那么malloc的需求是什么？如果所有元素都为零，那么for循环总是会在fp = _iob + 3处断开吗？
其次在main（）函数中，提到的模式是＆＃34; r＆＃34;但是后面使用的系统调用是write（），尽管它没有显示错误，但为什么这可能呢？

第三，编译器在编译此代码时不显示错误或警告，但显示一个对话框，指出＆＃34; f_open.exe已停止工作＆＃34;即使我们写了malloc行（或没有），或者即使有＆＃34; r＆＃34;模式或＆＃34; w＆＃34;模式。那有什么不对？

#include<fcntl.h>
#include<unistd.h>
#include<stdlib.h>
#define PERM 0644
#define EOF (-1)
#define BUFSIZE 1024
#define OPEN_MAX 20

typedef struct _iobuf{
    int cnt;
    char *ptr;
    char *base;
    int flag;
    int fd;
} myFILE;

enum _flags {
    _READ   = 01,
    _WRITE  = 02,
    _UNBUF  = 04,
    _EOF    = 010,
    _ERR    = 020
};

myFILE _iob[OPEN_MAX]={
    {0, (char *) 0, (char *) 0, _READ, 0 },
    {0, (char *) 0, (char *) 0, _WRITE, 1 },
    {0, (char *) 0, (char *) 0, _WRITE | _UNBUF, 2 }
};

#define stdin (&_iob[0])
#define stdout (&_iob[1])
#define stderr (&_iob[2])

#define getc(p)     ( --(p)->cnt>=0 ? (unsigned char) *(p)->ptr++ : _fillbuf(p) )

int _fillbuf(myFILE *fp)
{
    int bufsize;

    if((fp->flag & (_READ|_EOF|_ERR))!=_READ)
        return EOF;

    bufsize=(fp->flag & _UNBUF)? 1 : BUFSIZE;

    if(fp->base==NULL)
        if((fp->base=(char *)malloc(bufsize))==NULL)
            return EOF;

    fp->ptr=fp->base; 
    fp->cnt=read(fp->fd, fp->ptr, bufsize);

    if(--fp->cnt<0){
        if(fp->cnt == -1)
            fp->flag |= _EOF;
        else
            fp->flag |= _ERR;
        return EOF;
    }
    return (unsigned char) *fp->ptr++;  
}

myFILE *myfopen(char *name, char *mode)
{
    int fd;
    myFILE *fp;

    if(*mode!='r' && *mode!='w' && *mode!='a')
          return NULL;
    for(fp=_iob; fp<_iob+OPEN_MAX; fp++)
        if((fp->flag & (_READ | _WRITE))==0)
            break;

    if(fp>=_iob+OPEN_MAX)
        return NULL;

    if(*mode=='w')
         fd=creat(name, PERM);
    else if(*mode=='a'){
        if((fd=open(name, O_WRONLY, 0))==-1)
            fd=creat(name, PERM);   
        lseek(fd, 0L, 2);
    } else
        fd=open(name, O_RDONLY, 0);

    if(fd==-1)
        return NULL;

    fp->fd = fd;
    fp->cnt = 0;
    fp->base = NULL;
    fp->flag = (*mode=='r')? _READ : _WRITE;

        return fp;    
    } 

int main(int argc, char *argv[])
{
    myFILE *fp;
   int c;

    if((fp=myfopen(argv[1], "r"))!=NULL)
        write(1, "opened\n", sizeof("opened\n"));

     while((c=getc(fp))!=EOF)
          write(1, &c, sizeof(c));

    return 0;
}

提供的解决方案是：

myFILE *fp;

if(*mode!='r' && *mode!='w' && *mode!='a')
      return NULL;
for(fp=_iob; fp<_iob+OPEN_MAX; fp++)
    if((fp->flag & (_READ | _WRITE))==0) // marked line
        break;

当您到达标记的行时，您尝试取消引用fp指针。由于它（可能但不是肯定）初始化为零（但我应该说为NULL），因此您将取消引用空指针。繁荣。段错误。

这是你需要改变的地方。

myFILE *fp = (myFILE *)malloc(sizeof(myFILE));

确保#include使用malloc。

此外，您的关闭函数应该稍后释放（）您的myFILE以防止内存泄漏。

正如您所看到的，上面是关联问题的答案：segmentation fault on c K&R fopen and fillbuf

Answer 1

其他问题的诊断是假的

正如评论中所指出的，other question中的诊断是假的。特别是，myfopen()中的循环读取：

for (fp =_iob; fp <_iob + OPEN_MAX; fp++)
    if ((fp->flag & (_READ | _WRITE)) == 0)
        break;

完全正确。它遍历数组_iob的元素，并且永远不会遇到声明的空指针。缓冲区_iob初始化为前三个元素;其余的都是零。

可能的原因

此程序崩溃的最可能原因是：

没有为文件名提供参数。
无法打开提供的名称进行阅读。

main()显示的代码是：

int main(int argc, char *argv[])
{
    myFILE *fp;
   int c;

    if((fp=myfopen(argv[1], "r"))!=NULL)
        write(1, "opened\n", sizeof("opened\n"));

     while((c=getc(fp))!=EOF)
          write(1, &c, sizeof(c));

    return 0;
}

代码不会检查这些常见问题。它应该更像是：

int main(int argc, char *argv[])
{
    myFILE *fp;
    int c;

    if (argc != 2)
    {
        static const char usage[] = "Usage: mystdio filename\n";
        write(2, usage, sizeof(usage)-1);
        return 1;
    }

    if ((fp = myfopen(argv[1], "r")) == NULL)
    {
        static const char filenotopened[] = "mystdio: failed to open file ";
        write(2, filenotopened, sizeof(filenotopened)-1);
        write(2, argv[1], strlen(argv[1]));
        write(2, "\n", 1);
        return 1;
    }

    write(1, "opened\n", sizeof("opened\n"));

    while ((c = getc(fp)) != EOF)
        write(1, &c, sizeof(c));

    return 0;
}

您必须添加#include <string.h>，因为它使用strlen()。通过文件描述符I / O报告错误是笨拙的，但是此代码排除了使用<stdio.h>中的常规函数。该程序包含<stdlib.h>，因此可以编写exit(EXIT_FAILURE);而不是return 1;，但当前函数为main()时，最终结果相同 - 尽管不是函数（直接或间接）来自main()。

关于c程序中的myfopen（）

1 个答案:

其他问题的诊断是假的

可能的原因