编写syslog封装时vsprintf触发Segmentation Fault的问题排查

首先，你的段错误根源并非vsprintf调用本身，而是后续sprintf(entry, ...)中的野指针问题——char *entry;只声明了指针，但没有为它分配任何有效内存空间，直接用sprintf往这个未初始化的指针写入数据，会导致非法内存访问，触发段错误。你调试时误以为错误出在vsprintf，大概率是栈帧内存布局导致错误延迟上报，或是gdb栈回溯显示有偏差，但核心问题始终是entry指针未初始化。

具体问题分析 #

看你的代码片段：

char buffer[1000];
char *entry; // 未初始化的野指针，指向随机内存地址
va_list args;
va_start(args, format);
vsprintf(buffer, format, args); // 这里实际是正常的，buffer是栈上的有效数组
va_end(args);
sprintf(entry, "%s:%d - %s - %s",filename, lineNumber, functionName, buffer); // 这里尝试往野指针指向的地址写数据，直接触发段错误

修复方案 #

这里提供两种简单的修复方式，同时建议替换不安全的vsprintf/sprintf为vsnprintf/snprintf，避免缓冲区溢出风险：

方案1：使用栈上数组（简单高效）

直接把entry换成栈上的字符数组，合并日志前缀与格式化内容：

void logDebugFunction (int lineNumber, char* filename, const char* functionName, char* format, ...) {
    if (LOG_DEBUG >= argPtr->logLevel) {
        char logEntry[1024]; // 预留足够空间容纳前缀+格式化内容
        va_list args;
        va_start(args, format);
        // 先写入日志前缀
        int prefixLen = snprintf(logEntry, sizeof(logEntry), "%s:%d - %s - ", filename, lineNumber, functionName);
        // 确保前缀写入成功且有剩余空间，再追加格式化内容
        if (prefixLen > 0 && prefixLen < sizeof(logEntry)) {
            vsnprintf(logEntry + prefixLen, sizeof(logEntry) - prefixLen, format, args);
        }
        va_end(args);
        syslog(LOG_MAKEPRI(0, LOG_DEBUG), "%s", logEntry);
        if (argPtr->verbose) {
            printf("%s\n", logEntry);
        }
    }
}

方案2：动态分配内存（适合超长日志）

如果担心栈数组大小不够，可以用动态内存分配，记得用完释放：

void logDebugFunction (int lineNumber, char* filename, const char* functionName, char* format, ...) {
    if (LOG_DEBUG >= argPtr->logLevel) {
        va_list args;
        va_start(args, format);
        // 先计算格式化内容的长度
        int fmtLen = vsnprintf(NULL, 0, format, args);
        va_end(args);
        
        // 计算总长度：前缀长度 + 格式化内容长度 + 1（字符串终止符）
        int prefixLen = snprintf(NULL, 0, "%s:%d - %s - ", filename, lineNumber, functionName);
        int totalLen = prefixLen + fmtLen + 1;
        
        char *logEntry = malloc(totalLen);
        if (logEntry != NULL) {
            // 拼接前缀
            snprintf(logEntry, totalLen, "%s:%d - %s - ", filename, lineNumber, functionName);
            // 追加格式化内容
            va_start(args, format);
            vsnprintf(logEntry + prefixLen, totalLen - prefixLen, format, args);
            va_end(args);
            
            syslog(LOG_MAKEPRI(0, LOG_DEBUG), "%s", logEntry);
            if (argPtr->verbose) {
                printf("%s\n", logEntry);
            }
            free(logEntry);
        }
    }
}

额外优化建议 #

1. 坚持使用安全的打印函数：vsprintf/sprintf没有缓冲区大小检查，极易导致缓冲区溢出，建议始终用vsnprintf/snprintf替代。
2. 校验日志级别合法性：确保LOG_DEBUG和argPtr->logLevel的取值符合syslog的级别定义（如LOG_DEBUG、LOG_INFO、LOG_WARNING等）。
3. 保障全局指针安全：确认argPtr在调用logDebugFunction前已经正确指向有效的arguments结构体，避免空指针访问。

内容的提问来源于stack exchange，提问作者RobbG