使用strcat拼接命令行参数异常，为何未显示完整预期内容？

问题根源 #

• 直接修改argv字符串触发未定义行为：
  程序启动后，argv指向的字符串通常存储在只读内存区域，或多个字符串在内存中连续排列。直接调用strcat修改argv[i]会导致两种问题：要么触发内存写保护崩溃，要么越界覆盖相邻的argv元素。你输入的-l参数就是被这种覆盖破坏的，所以最终argv[1]里看不到该内容。
• 循环逻辑与条件判断错误：
  你的条件是判断「当前参数不是分号」就执行拼接，但实际应该判断「下一个参数不是分号」才继续拼接当前命令的参数。同时，每次拼接完当前和下一个参数后，没有跳过已处理的下一个参数，导致循环重复处理被破坏的元素，进一步加剧内存混乱。

以你的输入参数序列ls、-l、-a、.、;、date为例：

1. 当i=1时，执行strcat(argv[1], " ")和strcat(argv[1], argv[2])，直接覆盖了argv[2]（-l）所在的内存，使其变成无效值。
2. 当i=2时，处理的已经是被破坏的argv[2]，后续拼接操作继续打乱内存，最终导致argv[1]的内容变成混乱的ls -a . 。

修正思路 #

1. 使用动态缓冲区存储拼接结果：
   不要直接修改argv，用malloc/realloc创建足够大的字符串缓冲区，专门用来拼接每个命令的参数，避免破坏原有的argv数据。
2. 重构循环逻辑：
   • 遍历argv时，为每个以分号分隔的命令单独拼接参数。
   • 遇到分号时，结束当前命令的拼接，开始处理下一个命令。
   • 每次拼接完一个参数后，自动跳过已处理的参数，避免重复操作。

示例修正代码 #

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

int main(int argc, char **argv) {
    int i = 1;
    while (i < argc) {
        // 初始化当前命令的缓冲区
        char *cmd = malloc(1);
        *cmd = '\0';
        
        // 拼接当前命令的所有参数（直到遇到分号或参数结束）
        while (i < argc && strcmp(argv[i], ";") != 0) {
            size_t new_len = strlen(cmd) + 1 + strlen(argv[i]) + 1;
            cmd = realloc(cmd, new_len);
            
            if (strlen(cmd) > 0) {
                strcat(cmd, " ");
            }
            strcat(cmd, argv[i]);
            i++;
        }
        
        // 输出当前命令（后续可替换为exec系列调用执行）
        printf("当前命令：%s\n", cmd);
        free(cmd);
        
        // 跳过分号，进入下一个命令的处理
        if (i < argc && strcmp(argv[i], ";") == 0) {
            i++;
        }
    }
    return 0;
}

调用命令./program ls -l -a . \; date后，输出结果为：

当前命令：ls -l -a .
当前命令：date

额外注意事项 #

• 执行命令时，优先使用execvp这类直接传递参数数组的函数，比拼接成字符串后调用system更安全，能避免shell注入风险。
• 动态分配的内存务必用free释放，避免内存泄漏。

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