search

karawin / Ka-Radio32

esp32 version of Ka-Radio (https://github.com/karawin/Ka-Radio) by jp Cocatrix
516 stars 155 forks source link

Sizeof shoutcast_info > 256 vs getStation() #153

Closed bazooka07 closed 4 years ago

bazooka07 commented 4 years ago

Sorry but in french is more easy for us.

La liste des stations est un tableau d'enregistrements de 73+116+64+8+16=277 octets chacun (voir fichier main/include/eeprom.h). la fonction getStation() ne lit qu'un bloc de 256 octets (voir fichier main/include/eeprom.c). Cela ne me semble pas cohérent. Si une station a un "domain", un "file" et un "name" au maximum de leurs tailles, il y a certainement un risque de corruption de données. Il serait peut-être préférable de réduire la taille possible de "domain" et "file" pour être sûr que la taille d'un enregistrement soit inférieur à 256 octets.

Merci de m'éclairer sur ce point, je ne suis pas un expert en esp32.

karawin commented 4 years ago 
int8_t ovol; // offset volume
uint16_t port;  //port

8 bits soit 1 octet etc... Soit 1+2 et non 8+16

Donc 256 Bye

bazooka07 commented 4 years ago

Oups, effectivement je mélange les bits et les bytes. Donc 73+116+64+1+2=256 le compte est bon.

Désolé pour le dérangement.

karawin commented 4 years ago

Pas de pb, avec la somme de code ecrit, une bétise est vite arrivée. Autant vérifier, merci