请教一个关于指针的问题。

changyl123

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

int Get_Config(char *lpFileName,char *SerchBuf,char *szTmp)
{
    FILE *stream;
    char pBuf[256]="";
    char *p;
    char *ptoken = NULL;
   
   if((stream=fopen(lpFileName,"rt"))== NULL )   
  {
printf("opne file is error\n");
        return -1;
  }
while(!feof(stream))
{
memset(pBuf, 0x00, 256);
fgets(pBuf, 256, stream);
if(strstr(pBuf,SerchBuf))
{
p = strchr(pBuf, '=');
strncpy(szTmp, p + 1, sizeof(pBuf));
printf("this is very important data %s\n",szTmp);
break;
}
else
continue;
    }
    fclose(stream);
    return 1;
}


int main()
{
    char lpFileName[128]="Config.ini";
    char szTmp[128];
   
    memset(szTmp, 0x00, 128);
    if(Get_Config(lpFileName,"TMP_LEN=",szTmp)==-1)
    {
       printf("get config error\n");
    }
    printf("this TMP_LEN is %s\n",szTmp);

   getchar();
    return 0;   
}

我执行这个程序的时候，能够正确的读取一次配置文件的内容，但是第2次调用上面的这个Get_Config函数，就不能正常打开lpFileName文件了。这时显示为opne file is error。请问各位大虾扎个整哦？

yexilove

打开文件后要把它关掉。
fclose(stream);

changyl123

我的代码中加入了这行的。
不过还是只能执行一次，再次调用就不能打开文件。

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
-------------------------------------
这一行的问题，szTmp你只定义了128字节大小，拷贝用了sizeof(pBuf)=256,产生越界操作，把文件名区域都覆盖了，没有文件名，第二次当然打不开文件了。

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
--------------------------
这一行有问题，sizeof(pBuf)=256,而你的szTmp只有128字节大小，产生越界操作，把lpFileName里的内容覆盖了，再次调用函数时，没有文件名，打开文件当然出错了。

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
越界操作，冲了文件名

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
越界操作，覆盖了文件名，再次打开时就出错了。

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
越界操作，覆盖了文件名。

qingclie

不太可能哦

gogoandsoso

strncpy(szTmp, p + 1, sizeof(pBuf));
这行代码有错，越界了，文件名可能被覆盖了。