使用getopt结合atoi时触发Segmentation Fault（段错误）

看起来你的段错误问题出在两个细节的结合上：--port长选项被定义为optional_argument，但你的代码完全没处理用户未提供端口参数的场景，同时当optarg为NULL时直接传给atoi()，触发了空指针访问，导致段错误。

具体问题分析 #

1. 长、短选项行为冲突：你把--port定义为optional_argument（可选参数），但对应的短选项-p却加了冒号（"f:p:h"），这表示-p必须带参数，长、短选项的行为不一致，很容易混淆用户的使用方式。
2. 未处理optarg为NULL的情况：当用户使用--port但不带任何参数时，getopt_long会把optarg设为NULL。而atoi(NULL)会尝试读取空指针指向的内存，直接触发段错误。
3. atoi的转换检查不够严谨：虽然你写了if (!port)来判断无效端口，但atoi在转换非数字字符串时会返回0，而0本身是合法的端口（只是你不需要），这种判断逻辑可能会误杀合法输入，同时也没覆盖optarg为NULL的极端情况。

解决方案 #

根据你的需求（端口是必填还是可选），可以选择以下两种修复方式：

方式一：把端口设为必填参数（推荐）

如果端口必须由用户指定，直接把长选项的optional_argument改成required_argument，这样getopt_long会强制用户给--port传参数，避免optarg为NULL的情况：

static struct option long_options[] = {
    {"file", required_argument, 0, 'f'},
    {"port", required_argument, 0, 'p'}, // 修改为必填参数
    {"help", no_argument, 0, 'h'},
    {NULL, 0, NULL, 0}
};

同时用更严谨的strtol替代atoi，准确判断转换是否成功：

case 'p': {
    char *endptr;
    long port_long = strtol(optarg, &endptr, 10);
    // 检查：转换失败、端口不在合法范围（1-65535）
    if (endptr == optarg || port_long < 1 || port_long > 65535) {
        printf("not a valid port\n");
        exit(1);
    }
    port = (int)port_long;
    break;
}

方式二：保留端口为可选参数（使用默认值）

如果你希望端口可选（没传参时用默认的PORT），必须先判断optarg是否为NULL，再做后续处理：

case 'p':
    if (optarg == NULL) {
        // 未传参，使用默认端口
        port = PORT;
    } else {
        char *endptr;
        long port_long = strtol(optarg, &endptr, 10);
        if (endptr == optarg || port_long < 1 || port_long > 65535) {
            printf("not a valid port\n");
            exit(1);
        }
        port = (int)port_long;
    }
    break;

⚠️ 注意：当长选项是optional_argument时，用户必须用--port=123的形式传参，如果写成--port 123，getopt_long会把123当成下一个独立参数，此时optarg仍然是NULL，这是很容易踩的坑。

总结 #

你的核心问题是忽略了optarg可能为NULL的情况，导致atoi访问空指针引发段错误。通过调整长选项的参数类型，或者增加NULL检查，再配合strtol做严谨的数字转换，就能彻底解决这个问题。

备注：内容来源于stack exchange，提问作者Udeshya D.