c++ warning: sscanf() without field width limits can crash with huge input data. [invalidscanf]

字符串没有宽度限制的最大危害是容易被缓冲区攻击。

以下代码中 sscanf 会报 "没有字段宽度限制的sscanf()可能会在输入大量数据时崩溃。" 错误。

double GetRamUtilization() {
    FILE *file_fd = NULL;
    Memory_t memory;
    char buf[1024]{0};

    file_fd = ::fopen("/proc/meminfo", "r");
    if (file_fd == NULL) {
        ::perror("fopen /proc/meminfo error");
        return 0;
    }

    ::fgets(buf, sizeof(buf), file_fd);

    ::sscanf(buf, "%s %ld ", memory.name, &memory.total);

    ::fgets(buf, sizeof(buf), file_fd);
    ::sscanf(buf, "%s %ld ", memory.name2, &memory.free);

    ::fgets(buf, sizeof(buf), file_fd);
    ::sscanf(buf, "%s %ld ", memory.name3, &memory.available);

    ::fclose(file_fd);
    file_fd = NULL;

    return (double)(100.0 * (memory.total - memory.available)) / (memory.total);
}

加宽度限制即可,修改如下:

double GetRamUtilization() {
    FILE *file_fd = NULL;
    Memory_t memory;
    char buf[1024]{0};

    file_fd = ::fopen("/proc/meminfo", "r");
    if (file_fd == NULL) {
        ::perror("fopen /proc/meminfo error");
        return 0;
    }

    ::fgets(buf, sizeof(buf), file_fd);

    ::sscanf(buf, "%24[^-] %ld ", memory.name, &memory.total);

    ::fgets(buf, sizeof(buf), file_fd);
    ::sscanf(buf, "%24[^-] %ld ", memory.name2, &memory.free);

    ::fgets(buf, sizeof(buf), file_fd);
    ::sscanf(buf, "%24s %ld ", memory.name3, &memory.available);

    ::fclose(file_fd);
    file_fd = NULL;

    return (double)(100.0 * (memory.total - memory.available)) / (memory.total);
}

主要是%s要加宽度限制,%24[^-]或%24s。

  • 5
    点赞
  • 4
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值