シンプルなロガークラスvsnprintfが読み取りアクセス違反でクラッシュする

Question

シンプルなprintfスタイルのロガーを実装しているときに、vsnprintfクラッシュが発生しました。これは、私がロガーユーティリティを呼び出す方法です：

LoggerUtil->LogInfo("Whatever info here %s", "just a test!"); 

これは、可変数の引数を持つ関数を呼び出します。アイデアは私fmtを変更する必要があるので、フォーマット文字列に余分な情報を追加することです：主な問題を把握することはできません

std::string LoggerUtil::LogClientInfo(const char* fmt) 
{ 
    return "Some info here %s"; 
} 

void LoggerUtil::LogInfo(const char* fmt, ...) 
{ 
    std::string formatStr = LogClientInfo(fmt); // returns "Some info here %s" just for testing altering the format string 
    const char* format = formatStr.c_str(); // checked memory and its '\0' terminated string 
    va_list arg_list; 
    va_start(arg_list, format); 
    Logger::InfoVA(format, arg_list); 
    va_end(arg_list); 
} 

void Logger::InfoVA(const char* fmt, va_list arg_list) 
{ 
    Log(Priority_Info, fmt, arg_list); 
} 

void Logger::Log(Priority priority, const char* fmt, va_list args) 
{ 
    char str[MaxLogEntrySize]; 
    memset(str,0,MaxLogEntrySize*sizeof(char)); 
    vsnprintf(str,MaxLogEntrySize-1, fmt, args); // CRASH :(
    ... 
} 

を、FMTと焼戻しではないことは、問題を解決しますが、それはオプションではありません。

void LoggerUtil::LogInfo(const char* fmt, ...) 
{ 
    va_list arg_list; 
    va_start(arg_list, fmt); 
    Logger::InfoVA(fmt, arg_list); 
    va_end(arg_list); 
} 

私はここで何が欠けていますか？

Answer 1

LogClientInfo()を使用して新しいフォーマット文字列を作成した後、2番目のパラメータに間違った入力値をva_start()に渡しています。ローカルのformat変数を渡していますが、代わりにfmt引数をLogInfo()に渡す必要があります。 Logger::InfoVA()を呼び出すときに別の書式文字列を使用しているだけで、入力書式の値の格納場所は変わりません。 va_start()は、指定された引数に対する次関数の引数を指すようにva_listを構成し、fmt関数の引数ではなく、ローカルformat変数である必要があり、この場合には：

void LoggerUtil::LogInfo(const char* fmt, ...) 
{ 
    std::string formatStr = LogClientInfo(fmt); 
    va_list arg_list; 
    va_start(arg_list, fmt); // <-- use fmt here ! 
    Logger::InfoVA(formatStr.c_str(), arg_list); 
    va_end(arg_list); 
}