PointCloudLibrary/pcl

Check if read succedded before reading again/allocating memory

Open

#3.606 aberto em 29 de jan. de 2020

Ver no GitHub
 (16 comments) (0 reactions) (0 assignees)C++ (4.506 forks)batch import
good first issuehelp wantedkind: bugneeds: pr merge

Métricas do repositório

Stars
 (9.023 stars)
Métricas de merge de PR
 (Mesclagem média 8d 7h) (3 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Revise o link do Codacy para todas as 273 instâncias de validação de leitura ausente. Para cada arquivo, como features/src/narf.cpp, adicione uma verificação após a leitura do fluxo de entrada para confirmar o sucesso antes de alocar memória ou processar os dados. Considere criar uma função utilitária para realizar a verificação de forma consistente. Certifique se de que as alterações sejam compiladas e passem nos testes existentes.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
GitC++PCL basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70