PointCloudLibrary/pcl

Check if read succedded before reading again/allocating memory

Open

#3606 aperta il 29 gen 2020

Vedi su GitHub
 (16 commenti) (0 reazioni) (0 assegnatari)C++ (4506 fork)batch import
good first issuehelp wantedkind: bugneeds: pr merge

Metriche repository

Star
 (9023 star)
Metriche merge PR
 (Merge medio 8g 7h) (3 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Rivedi il link di Codacy per tutte le 273 istanze di mancata validazione della lettura. Per ogni file, come features/src/narf.cpp, aggiungi un controllo dopo la lettura dal flusso di input per verificare il successo prima di allocare memoria o elaborare i dati. Considera la creazione di una funzione di utilità per eseguire il controllo in modo coerente. Assicurati che le modifiche compilino e superino i test esistenti.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
GitC++PCL basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70