ArduPilot/ardupilot

nmea output code should use fixed buffer and fix snprintf issues

Aperta

#12.514 aperta il 8 ott 2019

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)C++ (15.603 fork)batch import
BUGLibrarygood first issue

Metriche repository

Star
 (9336 stelle)
Metriche merge PR
 (Merge medio 26g 16h) (166 PR mergiate in 30 g)

Descrizione

The AP_NMEA_Output code should allocate buffers once for max size of strings, and also needs to fix the use of snprintf for the lng_string to ensure it doesn't end up with an unterminated string in %s for next printf

Guida contributor

Direzione di ricerca
Esamina il codice AP NMEA Output, in particolare l'allocazione del buffer e le chiamate snprintf. Il problema menziona l'uso di buffer fissi per le dimensioni massime delle stringhe e la correzione dello snprintf di lng string per evitare stringhe non terminate. Rivedi l'implementazione attuale in libraries/AP NMEA Output/. Controlla i commenti per eventuali indicazioni aggiuntive. Assicurati che la correzione sia testata con scenari di output NMEA.
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.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65