[DEV-CPP]ARCHIVAGE DES LIENS D'UNE RECHERCHE SUR GOOGLE

Commentaire de max12 le 22/05/2007 12:15:10 administrateur CS

C'est bien jolie :D Mais juste un conseil d'ami, utilise FormatMessage avec GetLastError au lieu d'utiliser un switch avec toutes les erreurs possibles, tu va épargner du temps et réduire un peu la taille du .exe

A+

Commentaire de sebastienmz le 23/05/2007 11:52:22

en fait je programme depuis que j'ai 8 ans et comme je vais bientôt avoir 17 ça fera 9 ans que je programme j'ai commencer pas le pascal après le delphi pendant 3 ans (je continu) et  après le vb et maintenant le C, j'aurais voulu faire cette source en objet mais je trouve que l'objet (ATTENTION VOUS NE SEREZ PAS D'ACCORD AVEC MOI) c'est de la merde c'est pour cela que je suis passé au C, par exemple ceux qui se prétende "PROGRAMMEUR EN C++ (with classes) et qui utilise des malloc a la place de new frenchement C'EST NULL COMME LES CLASSES. Quand même le concept objet est BIEN mais a l'application c'est plus simple mais l'objet pour mes programmes que j'ai fais en Objet me prend plus de ligne que un bon vieux programme en BON C (peut être que c'est moi je problème avec l'objet mais je suis complètement allergique a ce concepte). Parlons de winsock c'est pas si complique car dans mon travaille je code beaucoup avec winsock et donc j'ai l'abitude, c'est mon travaille (je suis apprenant en sécu informatique) qui ma appris le C, franchement je trouve winsock super simple ce que je trouve plus dure c'est les memcpy ou le glut, opengl
bon pour cette source elle a été faite en 6 heure !
mon vrais problème c'est pour le recv j'aimerais d'abord savoir la taille que je reçois et après je prend les données, cela a été mon vrais problème
et j'ai de la chance, regarder le code source des recherches sur google il y a des balises pour indiquer le début des résultats et la fin donc cette source a pas été si compliquée a réaliser!    

Commentaire de Thaeron le 23/05/2007 22:37:41

Salut, j'ai porté ton code sous linux pour le tester.
Il est plein de maladresses (excusables vu que ça fait peu de temps que tu fais du C) mais aussi d'erreurs.
Je vais en détailler le maximum mais je vais sûrement en oublier (surtout que j'ai pas fini de tout corriger).
On commence avec le main :
rech = (CHAR *) malloc (len_argv1 + 1);
rech[len_argv1 + 1] = '\0';
(idem pour path)
C'est faux bien sûr ! tu écris au delà de ce que tu as alloué, la ligne correcte est : rech[len_argv1] = '\0';
Ensuite :
      memcpy (path, argv[2], len_argv2);
      path[len_argv2 + 1] = '\0';
      sprintf (path, "%s.txt", argv[2]);
(passons sur le +1) a quoi ça sert de faire un memcpy() si ensuite tu écrases tout par sprintf() ? Pire ! tu as alloué pour la taille de argv[2] et tu écris 4 caractères de plus, donc tu débordes !
Puis un appel à resolv() en lui passant un const char*, or dans resolv tu as "  gethostname (host, sizeof (host));" donc un coup de page de manuel et je vois que gethostname place le nom d'host dans la chaine qu'on lui file en premier paramètre soit ici un const ! Donc c'est très mal (sous linux ça plante le prog direct), en plus cette fonction (dans ton utilisation) est absolument inutile.

On arrive à la fonction connect () :
int re[MAX_RECV];
  for (i = 0; i <= MAX_RECV; i++)
    re[i] = 1;
Et bam un overflow de plus, si tu alloues x entiers, ils vont de 0 à x-1
donc la ligne correcte est for (i = 0;i < MAX_RECV; i++)
Ensuite tu copies des machins dans buffer:
memcpy (buffer, begin, strlen (begin));
Tu utilises memcpy() mais tu ne recopies pas le caractères '\0' signalant la fin de la chaîne du coup le strcat() suivant va planter.
Et si tu te dis "oui mais j'ai mis memset() à 0 avant" j'expliquerais après pourquoi tes memset() sont faux.
Bon après j'ai viré ton thread pendant le portage vu qu'il est totalement inutile, donc je ne sais pas si cette partie est buggée ou non.
Valgrind m'indique des problèmes dans ton parser mais il est un "peu" trop bordélique pour que je m'y penche.

Donc venons aux erreurs "pas grave mais presque" :
il faut utiliser sizeof() avec des tableaux que s'ils ne sont pas dynamiques, ce qui fait que       rech = (CHAR *) malloc (len_argv1 + 1);
      memset (rech, 0, sizeof (rech));
c'est faux car le sizeof() d'un pointeur c'est 4 (sur une architecture 32 bits). Donc tes memset() ne vont mettre à 0 que 4 octets.

Quelques conseils :
si tu veux vraiment mettre à 0 ta chaine allouée : utilise calloc() qui va le faire très bien (ça va malloc + memset). si tu veux dupliquer une chaine utilise strdup() qui malloc() + strcpy(). Lorsque tu veux copier une chaine n'utilise pas memcpy() mais strcpy() et tu n'auras pas à mettre le '\0' de fin de chaine manuellement. Si tu veux comparer des chaines, ne fait pas caractère par caractère dans un if, utilise strcmp() ou strncmp().

Si tu veux faire une chaine directement intialisée et dans laquelle tu ne copieras pas de données, ne fait pas : char begin[1000] = "GET /search?q="; qui gaspille la mémoire mais tout simplement char begin[] = "GET /search?q="; le compilo mettra la taille tout seul comme un grand.
Ne met pas plein de variables globales ! Les variables globales c'est mal ! Du moins en abuser c'est mal.
Ne déclare pas n'importe où tes variables, en C elles ont des places bien définies pour une portée qui est aussi bien définie donc c'est soit en début de fichier, soit en début de fonction, soit en début de bloc mais pas en plein milieux d'un bloc (cas de wsaData dans ton main).
Pour finir n'utilise pas des threads si elles ne servent à rien, si tu lances un thread c'est pour continuer ensuite l'exécution et non pour attendre qu'il se finisse pour ensuite continuer l'exécution.

Derniers conseils (plus larges) : use et abuse des pages de manuels (surtout qu'elles sont meme en français), passe à linux pour pouvoir utiliser valgrind par exemple, il te permettra de vérifier que ton programme fonctionne correctement sans des bugs cachés.
J'espère que tu n'as pas mal pris mes remarques/conseils/corrections et que tu vas nous mettre à jour ce code par une nouvelle version toute propre et exempte de bugs.
Je te souhaite bon code.
Enjoy,
Thaeron

Commentaire de sebastienmz le 24/05/2007 11:52:53

Donc je vais corriger tous ça mais je comprend pas je calcule la taille des argv et ensuit j'alloue l'espace COMPREND PAS si tu peut un peut approfondir le sujet des malloc car j'ai pas compris
strdup()ça fait un moment que l'on m'en parle mais j'ai pas vraiment compris
Calloc je connais le nom mais rien de plus
A les thread J'ADORE CA C'EST UNE DES MEILLEURS PARTIE DU C, mais en faite j'ai mis des thread pour avoire un time out sur la reception du contenu car quand je l'ai fait je n'étais pas sur une bonne ligne et certain packet prennais trop de temps a arriver.
Pour la requete je vais corriger ça toute suite
Pour linux je suis en multi boot avec aussi plusieur vmware et pour les man je fait GOOGLE man la fonction
je te remercie pour ton aide je vais corriger tout ça ce soire !
MERCI !!!