PointCloudLibrary/pcl

Check if read succedded before reading again/allocating memory

Open

#3 606 ouverte le 29 janv. 2020

Voir sur GitHub
 (16 commentaires) (0 réactions) (0 assignés)C++ (4 506 forks)batch import
good first issuehelp wantedkind: bugneeds: pr merge

Métriques du dépôt

Stars
 (9 023 stars)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Examinez le lien Codacy pour les 273 instances de validation de lecture manquante. Pour chaque fichier, comme features/src/narf.cpp, ajoutez une vérification après la lecture du flux d'entrée pour confirmer la réussite avant d'allouer de la mémoire ou de traiter les données. Envisagez de créer une fonction utilitaire pour effectuer cette vérification de manière cohérente. Assurez vous que les modifications se compilent et passent les tests existants.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitC++PCL basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70