относительно myfopen () в программе c - PullRequest
Новая
       6

относительно myfopen () в программе c

0 голосов
/ 01 июня 2018

Мой вопрос касается этого вопроса: ошибка сегментации на c K & R fopen и fillbuf .

  1. В этом вопросе в функции myfopen () есть цикл for;Чего я не понимаю, так это того, что fp будет иметь значение, начинающееся с _iob, но я не понимаю, какие значения примет fp-> flag после первых трех значений (которые были исправлены).Будут ли они принимать нулевые значения, и еще одна вещь заключается в том, что в ответах (см. Ниже код) они говорят, что мы должны использовать malloc для предоставления пространства памяти для fp, но fp уже предоставил пространство памяти, потому что массив _iob имеет пространство иfp будет продолжать принимать адреса элементов массивов, так что для чего нужен malloc? Также, если все элементы равны нулю, цикл for будет всегда прерываться при fp = _iob + 3?
  2. Во-вторых, в main ()функция, упомянутый режим - "r", но системный вызов, используемый после слов, - write (), хотя он не показывает ошибку, но почему это возможно?

  3. В-третьих, компилятор не показывает ошибку илипредупреждение о компиляции этого кода, но показывает диалоговое окно, в котором говорится, что «f_open.exe перестал работать», и это остается тем же, даже если мы пишем строку malloc (или без), или даже если есть режим «r» или режим «w»,Так что же не так? 

    #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), вы разыменовываете нулевой указатель.Boom.Segfault.

Вот что вам нужно изменить. 

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

Обязательно #include для использования malloc.

Кроме того, ваша функция close должна позже освободить () ваш myFILEдля предотвращения утечек памяти.

Как вы можете видеть выше, ответ на связанный вопрос дан: ошибка сегментации на c K & R fopen и fillbuf

1 Ответ

0 голосов
/ 01 июня 2018

Диагноз в другом вопросе является поддельным

Как отмечается в комментариях, диагноз в другом вопросе является поддельным.В частности, цикл в myfopen(), который гласит: 

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

, является совершенно правильным.Он перебирает элементы массива _iob и никогда не встречает нулевой указатель, как заявлено.Буфер _iob инициализируется для первых трех элементов;остальные - все нули.

Возможные причины неполадок

Наиболее вероятные причины сбоя этой программы:

  1. Не указано аргумента для именифайл.
  2. Указанное имя не может быть открыто для чтения.

Код, указанный для 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().Сообщение об ошибках с помощью файлового дескриптора ввода-вывода является неуклюжим, но этот код исключает использование обычных функций из <stdio.h>.Программа включает в себя <stdlib.h>, поэтому можно было бы написать exit(EXIT_FAILURE); вместо return 1;, но чистый результат такой же, когда текущая функция равна main() - хотя не в функциях, вызываемых (прямо или косвенно) изmain().

...