PointCloudLibrary/pcl

Check if read succedded before reading again/allocating memory

Open

#3,606 创建于 2020年1月29日

在 GitHub 查看
 (16 评论) (0 反应) (0 负责人)C++ (4,506 fork)batch import
good first issuehelp wantedkind: bugneeds: pr merge

仓库指标

Star
 (9,023 star)
PR 合并指标
 (平均合并 8天 7小时) (30 天内合并 3 个 PR)

描述

Your Environment

  • PCL Version: HEAD

Context

Bad parsing is inevitable, specially if operations are performed on read data without validation

Expected Behavior

Graceful error detection

Current Behavior

Files: 273 instances. One sample provided:

Possible Solution

Sanity checks on input stream after reading before allocating memory

贡献者指南

研究方向
查看 Codacy 链接中所有 273 处缺失读取验证的实例。对于每个文件，例如 features/src/narf.cpp，在从输入流读取数据后添加检查，以在分配内存或处理数据之前验证读取是否成功。考虑创建一个实用函数来一致地执行此检查。确保更改能够编译并通过现有测试。
技术栈
cpp
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
GitC++PCL basics
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70