va_arg 的字符串分段错误

Question

我正在制作一个类似于 printf 的函数，它需要接收一个字符串和如下参数：

form("Integer %d, String %s", 54, "STRING");

并创建一个字符串 "Integer 54, String STRING"。我正在使用 stdarg.h 库，因为我的函数需要具有可变数量的参数，具体取决于字符串。

问题是我收到 Segmentation fault。我发现只有当我使用传递给 va_arg 的字符串 (char*) 执行 strlen 或 strcpy 时才会发生这种情况。这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
#include <string.h>

char* form(char *format, ...)
{
    va_list ap;char sign;int br=0,lasti,memo=0;char* help;
    int ints; float floats; double doubles; char chars; char* strings;
    va_start(ap,format);
    char* result=(char*)calloc(100,strlen(format));
    strcpy(result,format);
    for(int i=0; result[i] ;i++)
        if(result[i]=='%')
        {   
            switch (result[i+1])    
            {
                case 'd': {
                    ints=va_arg(ap,int);
                    int b,save=ints,dec=1,j;char *p=result+i;
                    for(b=0;save;b++) {save/=10;dec*=10;}
                    for(dec/=10,j=0; dec ; j++) { p[j]=((ints/dec)%10)+0x30; dec/=10; }
                    strcpy(result+i+b,format+i-memo+2);memo+=b-2;
                } break;

                case 'f': {
                    floats=va_arg(ap,double);

                } break;

                case 'l': {
                    doubles=va_arg(ap,double);

                } break;

                case 'c': {
                    chars=va_arg(ap,int);
                    result[i]=chars;
                } break;

                case 's': {
                    strings=va_arg(ap,char*);
                    strcpy(result+i+strlen(strings),format+i-memo+2);memo+=(strlen(strings)-2);
                } break;

                default: printf("Unknown type.\n"); break;
            }
            i=0;
        }

    return result;
}

int main()
{
    char a[100];
    scanf("%s",a);
    char*s=form("treci %s peti",a);
    printf("%s", s);

    printf("\n");
    free(s);
    return 0;
}

strlen 或 strcpy 可能导致段错误的唯一方法是字符串不是空终止的，但我的是。那么这里出了什么问题，我该如何解决？

编辑：添加代码。

Answer 1

仅针对 %s 个案例进行了测试

char* form(char *format, ...)
{
    va_list ap;char sign;int br=0,lasti,memo=0;char* help;
    int ints; float floats; double doubles; char chars; char* strings;
    va_start(ap,format);
    char* result=(char*)calloc(100,strlen(format));
//    strcpy(result,format);
    for(int i=0; format[i] ;i++)
    {
        if(format[i]=='%')
        {
            switch (format[i+1])
            {
                case 'd': {
                    ints=va_arg(ap,int);
                    int b,save=ints,dec=1,j;char *p=result+i;
                    for(b=0;save;b++) {save/=10;dec*=10;}
                    for(dec/=10,j=0; dec ; j++) { p[j]=((ints/dec)%10)+0x30; dec/=10; }
                    strcpy(result+i+b,format+i-memo+2);memo+=b-2;
                } break;

                case 'f': {
                    floats=va_arg(ap,double);

                } break;

                case 'l': {
                    doubles=va_arg(ap,double);

                } break;

                case 'c': {
                    chars=va_arg(ap,int);
                    result[i]=chars;
                } break;

                case 's': {
                    strings=va_arg(ap,char*);
                    strcpy(result+memo,strings);
                    memo+=strlen(strings);
                } break;

                default: printf("Unknown type.\n"); break;
            }
            i++;
        }
        else
        {
            result[memo++] = format[i];
        }
    }

    return result;
}

很多问题：

strcpy(result+i+strlen(strings),format+i-memo+2); 是对结果的处理格式。您需要的是将作为变量参数传递的字符串复制到结果字符串。此外 result+i+strlen(strings) 没有任何意义：你为什么要这样写？
每次找到格式说明符时都会出现 i=0 %：为什么？这是 分段错误 的根本原因。每次找到 % 并请求一个新的变量参数时，循环都会重新启动，但仍未检索到其他参数。
您必须有 2 个不同的索引来解析输入格式字符串和索引结果字符串，因为输出字符串可能具有不同的长度。

Answer 2

使用vsnprintf非常简单：

char *form(const char *format, ...)
{
    va_list va;
    va_start(va, format);

    int result = vsnprintf(NUll, 0, format, va);

    // Error checking
    if (result < 0)
        return NULL;

    // Here result is the number of bytes we need to allocate (excluding terminator)
    char *string = malloc(result + 1);

    // Now do the actual formatting
    vsnprintf(string, result + 1, format, va);

    return string;
}

重要提示：请记住 free 您 return 的字符串。

va_arg 的字符串分段错误

String segmentation fault with va_arg

c

string

variadic-functions

segmentation-fault