[C] bot irc

[b0fh]

Hello,

Le problème est probablement que recv(2) travaille sur des données arbitraires et ne rajoute donc pas le terminateur nul à la fin d'une ligne. Quand un message plus court que le précédent est reçu, il est réécrit par dessus, et strstr ne voit pas la limite entre les deux.

Code :

read_bytes = recv(sockfd, buf, 255, 0);
//check for error
buf[read_bytes] = '\0';

Note le 255 pour éviter une erreur off-by-one.

Ce code n'est pas fiable parce qu'il n'y a aucune garantie qu'une ligne entière soit disponible après un appel à recv(2), meme si ça sera le cas quasiment tout le temps. Pour faire proprement tu devrais avoir une première boucle qui accumule les résultats des appels à recv dans un buffer intermédiaire, cherche les terminateurs \n et déclenche une action le cas échéant, et copie le reste du buffer éventuel au début.

Ton implémentation de PING/PONG est incomplète, le PONG est sensé retourner l'argument envoyé avec PING. Le serveur serait en droit de te déconnecter pour ça.

Dernièrement, pour tes chaines de caractère constantes,

Code :

char blah[100] = "foobarbaz";

A l'intérieur d'une fonction, cause l'allocation du buffer sur la stack et une initialisation des données en dur dans le code de la fonction, alors que

Code :

char blah[] = "foobarbaz";

a au moins l'avantage d'utiliser un buffer exactement de la bonne taille, et que

Code :

char *blah = "foobarbaz";

place la chaine dans la section de données, et ne génère pas de code d'initialiastion.