PointCloudLibrary/pcl

Check if read succedded before reading again/allocating memory

Offen

#3.606 geöffnet am 29.01.2020

Auf GitHub ansehenContributor Guide
 (16 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (4.506 Forks)batch import
good first issuehelp wantedkind: bugneeds: pr merge

Repository-Metriken

Stars
 (9.023 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8T 7h) (3 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie den Codacy Link für alle 273 Instanzen fehlender Lesevalidierung. Fügen Sie für jede Datei, wie z. B. features/src/narf.cpp, nach dem Lesen aus dem Eingabestrom eine Prüfung hinzu, um den Erfolg zu überprüfen, bevor Speicher zugewiesen oder Daten verarbeitet werden. Erwägen Sie, eine Hilfsfunktion zu erstellen, um die Prüfung konsistent durchzuführen. Stellen Sie sicher, dass die Änderungen kompilieren und bestehende Tests bestehen.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitC++PCL basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70