Segfault jeté sur une ligne de code

Question

J'utilise this library for libgps et j'ai quelques problèmes pour le faire fonctionner correctement.

L'erreur de mon débogueur après qu'il dit segfault est:

Cannot find bounds of current function 

La ligne de code est ce lancer located in this file, en ligne 132.

uint8_t checksum= (uint8_t)strtol(strchr(message, '*')+1, NULL, 16); 

Je ne connais pas le contexte de cette du tout, et je ne sais pas pourquoi cela ne ferait pas de segfault.

Mon code:

#include <stdio.h> 
#include <stdlib.h> 
#include <gps.h> 
#include <math.h> 

/* 
* 
*/ 
int main(int argc, char** argv) { 

    // Open 
    gps_init(); 
    gps_on(); 

    loc_t data; 

    gps_location(&data); 
    printf("%lf %lf\n", data.latitude, data.longitude); 

    return (EXIT_SUCCESS); 
} 

La fonction gps_location() vous emmène dans gps.c et de là elle se heurte serial.c, une fois qu'il court:

void serial_readln(char *buffer, int len) 
{ 
    char c; 
    char *b = buffer; 
    int rx_length = -1; 
    while(1) { 
     rx_length = read(uart0_filestream, (void*)(&c), 1); 

     if (rx_length <= 0) { 
      //wait for messages 
      sleep(1); 
     } else { 
      if (c == '\n') { 
       *b++ = '\0'; 
       break; 
      } 
      *b++ = c; 
     } 
    } 
} 

Sur le break retourne à gps.c va dans:

switch (nmea_get_message_type(buffer)) { 

qui le prend en nmea.c pour nmea_get_message_type ci-dessus.

Il passe ensuite la ligne:

if ((checksum = nmea_valid_checksum(message)) != _EMPTY) 

le prendre jusqu'à: uint8_t checksum= (uint8_t)strtol(strchr(message, '*')+1, NULL, 16); qui est où l'erreur est.

À quoi est-ce dû?

Edit:

uint8_t est défini comme: typedef unsigned char uint8_t;

Answer 1

Segmentation fault est pas « jeté exception » en tant que tel, il est un problème émis matériel ("vous avez dit y aller, mais je n » t voir quelque chose nommé «là»). En ce qui concerne votre problème: que se passe-t-il lorsque le strchr() ne trouve pas le caractère spécifié? Je vous suggère de l'essayer et de le découvrir.

Answer 2

Le code avec lequel vous travaillez est horrible et n'a aucune vérification d'erreur nulle part. Donc, il peut aller haywire pour toute entrée inattendue. Cela pourrait aussi être une vulnérabilité de sécurité potentielle.

Pour résoudre ce cas particulier, modifiez le code:

if (!message) 
    return NMEA_CHECKSUM_ERR; // possibly `exit` or something, this shouldn't happen 

char *star = strchr(message, '*'); 
if (!star) 
    return NMEA_CHECKSUM_ERR; 

uint8_t checksum = strtol(star, NULL, 16); 

Le nmea_parse_gpgga et nmea_parse_gprmc ont également plusieurs instances d'un problème similaire.

Ces fonctions peuvent être acceptables s'il existe un analyseur ou une vérification d'expression rationnelle qui nettoie l'entrée avant d'appeler ces fonctions. Cependant, en fonction de votre question (je n'ai pas vérifié la base de code), il semble que les données sont passées directement de read ce qui est inexcusable.

La fonction segfaulting n'a pas été conçu pour traiter un message vide ou, en fait, aucun message ne correspond pas à la forme prévue.

Une autre erreur désastreuse est que la fonction serial_readln ne vérifie jamais qu'elle n'écrit pas au-delà de len.