fprintf也会core dump

liangzhishao

程序一跑就崩溃， 刚刚调试出引起崩溃的bug


修改代码后ok了


把

1. int detach_log(const char *filename, const char *p)
   
2. {
   
3.   FILE* fp;
   
4.   fp=fopen(filename,"a+");
   
5.   if(!fp) return -1;
   
6.   
   
7.   fprintf(fp,  p) ;
   
8.   fclose(fp);
   
9.   return 1;
   
10. }

复制代码

改成

1. int detach_log(const char *filename, const char *p)
   
2. {
   
3.   FILE* fp;
   
4.   fp=fopen(filename,"a+");
   
5.   if(!fp) return -1;
   
6.   
   
7.   fprintf(fp, "%s", p) ;
   
8.   fclose(fp);
   
9.   return 1;
   
10. }

复制代码

就ok了


就是把 fprintf(fp,  p)  改成  fprintf(fp, "%s", p)

我以前经常给printf族的函数 传一个 字符串指针，难道我一直都做错了？

难道printf("%s",  p_str)  才是安全的？

fcoolx

core dump是什么类型的原因？ 内存越界，还是其他什么？

maxxfire

是的，如果你确认传入的串不含格式，用fputs比较安全。。

sunlan

NAME

    fprintf, printf, snprintf, sprintf - print formatted output

SYNOPSIS

    #include <stdio.h>

    int fprintf(FILE *restrict stream, const char *restrict format, ...);
    int printf(const char *restrict format, ...);
    int snprintf(char *restrict s, size_t n,
           const char *restrict format, ...);
    int sprintf(char *restrict s, const char *restrict format, ...);

楼主为什么不查一下手册？

liangzhishao

Segmentation fault: 11 (core dumped)


signal 11 :  segmentation violation


差不多   我见过的崩溃都是第11信号  还没见过由其他原因引起的崩溃

liangzhishao

回复 4# sunlan


    手册当然看，不过这种简单的问题却忽略了，
      fprintf(FILE * restrict stream, const char * restrict format, ...);
   只是以前写printf写的习惯了，  忘了它要解释 format 的格式， 不好的习惯。