Incorrect usage of an automatic variable

mgll0120

#include    <stdio.h>

#define DATAFILE    "datafile"

FILE *
open_data(void)
{
    FILE    *fp;
    char    databuf[BUFSIZ];   /* setvbuf makes this the stdio buffer */

    if ((fp = fopen(DATAFILE, "r")) == NULL)
        return(NULL);
    if (setvbuf(fp, databuf, _IOLBF, BUFSIZ) != 0)
        return(NULL);
    return(fp);     /* error */
}
UNIX环境高级编程7.10最后有这样一段代码。在下愚昧，对fp进行写操作。然后查看datafile中的内容，没发现什么错误。

我的问题是：如何操作，可以看出自动变量databuf在open_data返回后的释放 对fp产生的“影响”。诚请各位指点。

nizvoo

你在setvbuf后對文件讀寫試驗下。

mgll0120

回复 2# nizvoo


    你是指：在open_data返回之后对filedata进行读写还是在setvbuf调用之后对filedata进行读写？
    前者我试过了，没发现问题。我现在就是想知道：如何操作可以看到问题。因为文章是说databuf是个局部变量，函数返回后会释放，会对fp的操作有影响。
    后者没有尝试，这不在我考虑的范围内。

    附上文中的话。
    The problem is that when open_data returns, the space it used on the stack will be used by the stack frame for the next function that is called. But the standard I/O library will still be using that portion of memory for its stream buffer. Chaos is sure to result. To correct this problem, the array databuf needs to be allocated from global memory, either statically (static or extern) or dynamically (one of the alloc functions).

nizvoo

你在這個函數裏頭嘗試。

starwing83

#include    <stdio.h>
#include    <string.h>

#define DATAFILE    "datafile"

FILE *
open_data(void)
{
    FILE    *fp;
    char    databuf[BUFSIZ];   /* setvbuf makes this the stdio buffer */

    if ((fp = fopen(DATAFILE, "r")) == NULL)
        return(NULL);
    if (setvbuf(fp, databuf, _IOLBF, BUFSIZ) != 0)
        return(NULL);
    return(fp);     /* error */
}
void foo() {
    FILE *dummy = 0;
    char databuf[BUFSIZ] = {};

    dummy = 0; // use it, or compiler will remove it.
    strncpy(databuf, "xhello", 5);
}

int main(void)
{
    char buf[BUFSIZ];

    FILE* fp = open_data();

    if (fp != NULL)
    {
        char a = fgetc(fp);
        foo();
        fgets(buf, BUFSIZ, fp);
        printf("%c%s\n", a, buf);
        fclose(fp);
    }

    return 0;
}


试试这个。需要有datafile文件，内容随意，就比如是foo吧。

mgll0120

回复 5# starwing83


    谢谢你的回复。貌似明白了一点。
   
    还有个问题想问一下，就是char a = fgetc(fp);这句话不是很明白。在FILE *fp = open_data();时不是已经打开了吗？为什么现在还  必须 加上这句才可以。