2

我有一个将二进制数据写入文件或stdout. 但是调用fwrite()有时会失败，除非我fflush(stderr)在stderr尝试fwrite.

这是正常行为，还是表明存在一些潜在的内存问题？调试很困难，因为一旦我尝试到数据fprint，“问题”就会消失。stderrfwritestdout

这是该函数的一个非常简化的版本。 

int writebinary(FILE *fpout, void *data, long ntowrite) {

    long i; 

    /* lets say we know the input data is double float */
    double *p = data; 

    /* including this test line makes the function work! */
    for(i=0; i<ntowrite;i++) fprintf(stderr,"data[%d]=%g\n",i,p[i]);

    /* attempt to write data[] as a single block of bytes */
    m= fwrite(data,(size_t)(ntowrite*sizeof(double)),1,fpout);

    if(m!=1) {
        fprintf(stderr,"Write error: %d\n",ferror(fpout));
        return(-1); 
    }
    else return(m);
}

任何智慧都值得赞赏:)

4

1 回答 1

0

这不是正常行为。
推荐以下。帖子中有许多值得关注的事情（见下文），但没有指出问题 - 但有些事情是错误的。

int writebinary2(FILE *fpout, void *data, long ntowrite) {
  size_t n = (size_t) ntowrite;
  double *dp = (double *) data;
  // pedantic checks
  if ((!fpout) || (!data) || (dp != data) || (ntowrite <= 0) || (n != ntowrite)) {
    fprintf(stderr,"Bad parameter\n");
    return(0);
  }
  clearerr(stderr);  // only needed for debug
  clearerr(fpout);
  /* attempt to write blocks of doubles */
  size_t m = fwrite(dp, sizeof(double), n, fpout);
  if (m != n) {
    fprintf(stderr,"Write error: %d\n",ferror(fpout));
    clearerr(fpout);
    return(-1);
  }
  if (fflush(fpout)) {
    fprintf(stderr,"Flush error: %d\n",ferror(fpout));
    clearerr(fpout);
    return(-1);
  }
  return(1);
}
  1. ntowrite s/b size_t，一个无符号整数。ntowrite = 0 可能会导致 m 混淆。ntowrite < 0将导致问题。
  2. m 未声明，假定为 size_t。
  3. 格式 "数据[%d]=%g\n" s/b "数据[%ld]=%g\n"
  4. 是/b size_t。
  5. (size_t)(ntowrite*sizeof(double))s/b ((size_t)ntowrite)*sizeof(double)。见#7。
  6. fprintf(stderr...不应该需要。这就是问题所在。
  7. fwrite(..., size, n ...)即使 size * n溢出也可以工作。比OP更安全。
于 2013-06-28T19:02:45.577 回答