c - 字符串操作中的双重错误

Question

如果传递了一个长字符串，则为以下代码获取双重免费。我尝试了各种各样的事情。如果我删除 free(s) 行，它就会消失。不确定为什么会这样。

void format_str(char *str1,int l,int o) {
    char *s = malloc(strlen(str1)+1);
    char  *s1=s, *b = str1;
    int i=0;
    while(*str1!='\0') {
        i++;
        *s1++=*str1++;

        if(i>=l) {
            if(*str1!=',') {
                continue;
            }
        *s1++=*str1++;
            *s1++='\n';
            for(i=0;i<o;i++) {
                *s1++=' ';
            }
            i = 0;
        }
    }
    *s1 = '\0';
    strcpy(b,s);
    free(s);
}

Answer 1

您可能没有在 s 中为要复制的数据量分配足够的空间。我不知道你的逻辑到底在做什么，但我看到了类似的东西

        *s1++=*str1++;
        *s1++='\n';

您要将多个字符复制到 s(通过 s1)以获取 str1 中的单个字符。

为了所有可计算的东西，使用更好的变量名!