使用vfprintf程序崩溃

夹心饼干 2014-07-23 05:41:49
gdb调试错误信息如下
#0 0x00000037fc8797c0 in strlen () from /lib64/libc.so.6
#1 0x00000037fc846b69 in vfprintf () from /lib64/libc.so.6
#2 0x0000000000412d98 in Logger::logva(const Logger::._78, const char *, va_list &) (this=0x657eb0, level=LEVEL_DEBUG, pattern=0x42f978 "初始化 %s:%u 成功", vp=...) at my_log.cpp:171
#3 0x0000000000412ecb in Logger::debug (this=0x657eb0, pattern=0x42f978 "初始化 %s:%u 成功") at my_log.cpp:266

相关源代码

struct tm *now;
time_t ltime;
char szName[64];

if (m_level > level) return;
time(<ime);
if (NULL == (now=localtime(<ime))) return;

m_lock.lock();

if (!m_file.empty())
{
if (m_hour != now->tm_hour)
{
if (NULL != fp_file)
{
fclose(fp_file);
}
m_hour = now->tm_hour;
snprintf(szName,sizeof(szName),"%s.%04d%02d%02d-%02d",m_file.c_str(),now->tm_year + 1900,now->tm_mon + 1,now->tm_mday, now->tm_hour);
fp_file = fopen(szName,"at");
truncate(m_file.c_str(), 0);
}
}

if (NULL != fp_console)
{
fprintf(fp_console,"%04d/%02d/%02d ",now->tm_year + 1900,now->tm_mon + 1,now->tm_mday);
fprintf(fp_console,"%02d:%02d:%02d ",now->tm_hour,now->tm_min,now->tm_sec);
}
if (NULL != fp_file)
{
fprintf(fp_file,"%04d/%02d/%02d ",now->tm_year + 1900,now->tm_mon + 1,now->tm_mday);
fprintf(fp_file,"%02d:%02d:%02d ",now->tm_hour,now->tm_min,now->tm_sec);
}

if (NULL != fp_console)
{
fprintf(fp_console,"[%s] ",m_name.c_str());
}
if (NULL != fp_file)
{
fprintf(fp_file,"[%s] ",m_name.c_str());
}

if (NULL != fp_console)
{
fprintf(fp_console," %s ==>", m_fileLine.c_str());
}
if (NULL != fp_file)
{
fprintf(fp_file," %s ==>", m_fileLine.c_str());
}

if (NULL != fp_console)
{
vfprintf(fp_console,pattern,vp);
fprintf(fp_console,"\n");
fflush(fp_console);
}
if (NULL != fp_file)
{
vfprintf(fp_file,pattern,vp); //这里崩溃
fprintf(fp_file,"\n");
fflush(fp_file);
}

m_lock.unlock();
...全文
3801 6 打赏 收藏 转发到动态 举报
写回复
用AI写文章
6 条回复
切换为时间正序
请发表友善的回复…
发表回复
夹心饼干 2014-07-23
  • 打赏
  • 举报
回复
2楼说的是对的,谢谢
mujiok2003 2014-07-23
  • 打赏
  • 举报
回复
if (NULL != fp_console)
    {
        vfprintf(fp_console,pattern,vp);
        fprintf(fp_console,"\n");
        fflush(fp_console);
    }
    if (NULL != fp_file)
    {
        vfprintf(fp_file,pattern,vp);            //这里崩溃
        fprintf(fp_file,"\n");
        fflush(fp_file);
    }
 
vp在fp_console和fp_file都不为NULL时被修改了两次, 确保vp对应了四个实参%s%u%s%u(因为pattern是"初始化 %s:%u 成功")
alofm 2014-07-23
  • 打赏
  • 举报
回复
引用 3 楼 alofm 的回复:
[quote=引用 2 楼 alofm 的回复:]

 if (NULL != fp_console)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_console,pattern,vp);
        fprintf(fp_console,"\n");
        fflush(fp_console);
    }
    if (NULL != fp_file)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_file,pattern,vp);            //这里崩溃
        fprintf(fp_file,"\n");
        fflush(fp_file);
    }

   vfprintf(fp_console,pattern,aq);
    vfprintf(fp_file,pattern,aq);         ----要用aq。
   
[/quote] -----------------打都不会字。。。。。。。。。。。
alofm 2014-07-23
  • 打赏
  • 举报
回复 1
引用 2 楼 alofm 的回复:

 if (NULL != fp_console)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_console,pattern,vp);
        fprintf(fp_console,"\n");
        fflush(fp_console);
    }
    if (NULL != fp_file)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_file,pattern,vp);            //这里崩溃
        fprintf(fp_file,"\n");
        fflush(fp_file);
    }

    vfprintf(fp_file,pattern,ap);         ----要用ap。

alofm 2014-07-23
  • 打赏
  • 举报
回复

 if (NULL != fp_console)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_console,pattern,vp);
        fprintf(fp_console,"\n");
        fflush(fp_console);
    }
    if (NULL != fp_file)
    {
       va_list aq;
       va_copy(aq,vp); ----两个vfprintf连续使用必须复制一份。
        vfprintf(fp_file,pattern,vp);            //这里崩溃
        fprintf(fp_file,"\n");
        fflush(fp_file);
    }
赵4老师 2014-07-23
  • 打赏
  • 举报
回复
仅供参考
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef WIN32
    #include <windows.h>
    #include <io.h>
#else
    #include <unistd.h>
    #include <sys/time.h>
    #include <pthread.h>
    #define  CRITICAL_SECTION   pthread_mutex_t
    #define  _vsnprintf         vsnprintf
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef WIN32
void Lock(CRITICAL_SECTION *l) {
    EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
    LeaveCriticalSection(l);
}
#else
void Lock(CRITICAL_SECTION *l) {
    pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
    pthread_mutex_unlock(l);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
    struct tm *now;
    struct timeb tb;

    if (NULL==pszFmt||0==pszFmt[0]) return;
    _vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
    ftime(&tb);
    now=localtime(&tb.time);
    sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
    sprintf(timestr,"%02d:%02d:%02d",now->tm_hour     ,now->tm_min  ,now->tm_sec );
    sprintf(mss,"%03d",tb.millitm);
    printf("%s %s.%s %s",datestr,timestr,mss,logstr);
    flog=fopen(logfilename1,"a");
    if (NULL!=flog) {
        fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
        if (ftell(flog)>MAXLOGSIZE) {
            fclose(flog);
            if (rename(logfilename1,logfilename2)) {
                remove(logfilename2);
                rename(logfilename1,logfilename2);
            }
        } else {
            fclose(flog);
        }
    }
}
void Log(const char *pszFmt,...) {
    va_list argp;

    Lock(&cs_log);
    va_start(argp,pszFmt);
    LogV(pszFmt,argp);
    va_end(argp);
    Unlock(&cs_log);
}
//Log}
int main(int argc,char * argv[]) {
    int i;
#ifdef WIN32
    InitializeCriticalSection(&cs_log);
#else
    pthread_mutex_init(&cs_log,NULL);
#endif
    for (i=0;i<10000;i++) {
        Log("This is a Log %04d from FILE:%s LINE:%d\n",i, __FILE__, __LINE__);
    }
#ifdef WIN32
    DeleteCriticalSection(&cs_log);
#else
    pthread_mutex_destroy(&cs_log);
#endif
    return 0;
}
//1-78行添加到你带main的.c或.cpp的那个文件的最前面
//81-85行添加到你的main函数开头
//89-93行添加到你的main函数结束前
//在要写LOG的地方仿照第87行的写法写LOG到文件MyLog1.log中

64,646

社区成员

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论,技术干货分享,前沿动态等
c++ 技术论坛(原bbs)
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题,如果涉及到代码请尽可能的格式化一下

试试用AI创作助手写篇文章吧