[C] Socket + envoie de commande

[notfound]

En fait, y'a un truc CAPITAL dans bordel ! C'est de mettre à la fin, le caractère 0x0d (CR, carrier return) sinon ça push pas le merdier et rien ne se passe ...

[Kiwazaru]

Exact, si on regarde le stream en "C Array" on vois bien :

Code C :


/* Le seul qui ne contient pas de CR */
char peer0_0[] = {
0x45, 0x53, 0x43, 0x2f, 0x56, 0x50, 0x2e, 0x6e,
0x65, 0x74, 0x10, 0x03, 0x00, 0x00, 0x00, 0x00 };

/* Tout les autres en contiennent */
char peer0_1[] = {
0x56, 0x45, 0x52, 0x3f, 0x0d };

char peer0_2[] = {
0x4c, 0x41, 0x4d, 0x50, 0x3f, 0x0d };

char peer0_3[] = {
0x50, 0x57, 0x53, 0x54, 0x41, 0x54, 0x55, 0x53,
0x3f, 0x0d };

char peer0_4[] = {
0x53, 0x4f, 0x55, 0x52, 0x43, 0x45, 0x3f, 0x0d };

char peer0_5[] = {
0x56, 0x45, 0x52, 0x3f, 0x0d };

char peer0_6[] = {
0x53, 0x4e, 0x4f, 0x3f, 0x0d };

char peer0_7[] = {
0x4d, 0x42, 0x53, 0x54, 0x41, 0x54, 0x55, 0x53,
0x3f, 0x0d };

char peer0_8[] = {
0x49, 0x4d, 0x50, 0x57, 0x52, 0x20, 0x4f, 0x46,
0x46, 0x0d };
 

Donc si tu veux que ça marche du coup, soit tu fait avec la technique j'envoie tout les paquets , soit tu fait la technique à gruik qui fait: Bonjour -> Shutdown

Code C :


/* Bonjour -> VP */
send(sock, "ESC/VP.net\x10\x03\x00\x00\x00\x00", len, 0);
recv(sock, reponse, MAX, 0);

/* Si on reçois une réponse "Bonjour" VP -> PC on essaye de l'éteindre */
if(strcmp(reponse, "ESC/VP.net\x10\x03\x00\x00\x20\x00") == 0){
      send(sock, "IMPWR OFF\x0d", len, 0);
      recv(sock, reponse, MAX, 0);
}

/* Si on reçois la requête disant que le VP a été éteint on affiche que VP est éteint */
if(strcmp(reponse, ":IMEVENT=0001 04 00000002 00000000 T1 F1\x0d\x3a:IMEVENT=0001 04 00000002 00000000 T1 F1\x0d\x3a") == 0){
      printf("[+] Vidéo-Projecteur Éteint !");
}
else{
      printf("[+] Erreur pendant la tentative !");
}
 

Je sais pas si le code est bon ou pas, j'ai enlevé les sprintf(); , ça servait pas à grand chose à part encombrer et ralentir l’exécution du programme

PS : Pas eu le temps d'édit, juste remplace STRCMP(); par MEMCMP();

[notfound]

Je paste mon code demain quand je retourne au boulot, il est fonctionnel

Edit : Code fonctionnel copié dans le 1er post !

[Kiwazaru]

Code C :


    printf("\nChoix : ");scanf("%d", &choix);
 

C'est un peu... :p

Code C :


   printf("\nChoix : ");
   fgets(choix, 10, stdin);
   fseek(stdin, 0, SEEK_END);
 

[notfound]

Un peu what ?

[Kiwazaru]

Bah scanf(); vérifie pas la taille de la chaîne envoyé donc si ton char choix[vaut 10] et que j'entre 012345678910AAAAAAAAAAAAAAAAAAAAAAAAAAA ça va planter :p. Certes c'est un code personnel mais c'est toujours bien de s'habituer :p

Là avec fgets(); tu peux limiter la chaîne , et faut vider après stdin qui sert de "buffer" , avec fseek(); donc :p

[notfound]

Ok ok je le saurai ! Thx

[gruik]

Bah scanf(); vérifie pas la taille de la chaîne envoyé donc si ton char choix[vaut 10] et que j'entre 012345678910AAAAAAAAAAAAAAAAAAAAAAAAAAA ça va planter :p.

ca vaut que pour un scanf %s, là le spécificateur %d fait qu'on le fera pas déborder
dans tous les cas on peut préciser plus finement avec scanf %10s par exemple

Code C :

int main (void) {
        int nombre = 0;
        printf ("entrer un nombre (4 chiffres max): ");
        scanf ("%4d", &nombre);
        printf ("nombre = %d\n", nombre);
        return 0;
}

Code BASH :

$ ./pouet
entrer un nombre (4 chiffres max): 1234567890
nombre = 1234

Certes c'est un code personnel mais c'est toujours bien de s'habituer :p

+1

[Kiwazaru]

Ah , je savais pas ça ! Sûrement trop d'idées reçues de dire que scanf(); == fonction horrible à oublier
Mais pourquoi ça existe pour les decimal et pas pour les chaînes de caractères? :o

[supersnail]

Ben si y'a le %10s qui limite la taille de l'entrée à 10 caractères.

Par contre comme partout, faut penser à vider le buffer avant de refaire un autre appel à scanf ,sous peine de bugs "bizarres".

[Kiwazaru]

Ah bah voilà, alors pourquoi tu me disais que scanf(); était à oublier? :p

[supersnail]

A cause des bugs bizarres que ça provoque (ie si tu rentres autre chose qu'un nombre quand t'as scanf("%d"), ta variable n'est pas modifiée, et le nombre reste non traité dans le buffer, et va donc "polluer" les appels suivants )

Démonstration avec le petit programme suivant:

Code C :

#include <stdio.h>

int main() {
        int age;
        char nom[11];
        printf ("Entrez votre âge: ");
        scanf ("%d", &age);
        printf ("Entrez votre nom: ");
        scanf ("%10s", nom);
        printf ("Vous vous appelez %s et vous avez %d ans\n", nom, age);
        return 0;
}
 

Qui, si dans le champ "âge", on rentre "tapz", retourne chez moi

Code :

$ ./wtf
Entrez votre âge: tapz
Entrez votre nom: Vous vous appelez tapz et vous avez -1217413120 ans

, ce qui est évidemment très loin du résultat attendu :'')

[Kiwazaru]

Yep effectivement :p
Oui donc en fait, c'est utilisable mais ça peut provoquer des erreurs de gestion donc fgets(); reste la meilleur solution pour avoir un programme fonctionnel et non "bugué"