socket 100% CPU

Salut@tous

Voilà la fonction que j'utilise pour savoir si un port est ouvert sur une ip:
DWORD WINAPI connection( LPVOID Param )
 {
 struct config * conf = (struct config *)Param;
 WSADATA WSAData;
 WSAStartup(MAKEWORD(2,0), &WSAData);
 conf->sin.sin_addr.s_addr = ntohl(conf->ip);
 conf->sin.sin_family = AF_INET;
 conf->sin.sin_port = ntohs(conf->port);
 conf->sock = socket(AF_INET,SOCK_STREAM,0);
 bind(conf->sock, (SOCKADDR *)&conf->sin, sizeof(conf->sin));
 if ( !connect(conf->sock, (SOCKADDR *)&conf->sin, sizeof(conf->sin)) )
       {
       FILE * fic;
    char retour[] = "\n";
       /* Ouverture d'un fichier */
       fic = fopen("resultats.dat", "a+");              
       /* Ecrire chaîne dans le fichier */
       fwrite(inet_ntoa(conf->sin.sin_addr), strlen(inet_ntoa(conf->sin.sin_addr)), 1, fic);
       fwrite(retour, strlen(retour), 1, fic);
       fclose(fic);
       }
    nthread++;
 closesocket(conf->sock);
 WSACleanup();
 CloseHandle(conf->hthread);
 free(conf);
 }

Mais dans le cas de mon scanner multithread basé sur le code :
http://www.cppfrance.com/code.aspx?ID=28177
J'obtiens un cpu 100%
J'imagine qu'il y a d'autres façon de savoir si un port est ouvert
J'espère que quelqu'un pourra me dire ce qui est à l'origine de cette surconsommation de ressources.

Je pense que c'est là que je dois intervenir:
bind(conf->sock, (SOCKADDR *)&conf->sin, sizeof(conf->sin));
 if ( !connect(conf->sock, (SOCKADDR *)&conf->sin, sizeof(conf->sin)) )

Regarde ici:
http://www.cppfrance.com/code.aspx?ID=11370

ciao...
BruNews, MVP VC++

Salut,
Dans ma source j'utilise l'api GetTcpTable(), fonction qui ne permet pas d'obtenir le PID du processus qui a ouvert le port (ca peut etre utile).

Cosmobob a posté une source plus récente qui permet de faire cela:
http://www.cppfrance.com/code.aspx?ID=24984

en fait je me demande si c'est pas plutot la fonction thread qui poserait problème.
Car même avec un thread de 1 ça monte à fond mon CPU

for (;startip <= endip;startip++)
   {
                      while ( !nthread );
    {
    conf = (struct config *) malloc(sizeof(struct config) );
    conf->ip = startip;
    conf->port = port;
    conf->hthread = CreateThread(NULL, 0, &connection, conf, 0, NULL);
    nthread--;
    }
   }

Le probleme des threads c'est que justement il te bouffe des ressources processeurs en + de ton programme et il faut savoir que les connexions reseaux sa peut etre super lourd a gerer.... Tu utilise toujours 100 threads a la fois ?? parce que si c'est le cas, cherche pas plus loin....  faudrait revoir le nombre de tread a la baisse au t'acheter une tour 4 proce bi-xeon avec hyperthreading et 10 go de ram ... lol
Non serieux, j'ai un p4 2,8e (o/c @ 3.33) et meme avec l'hyperthreading, j'ai certain programme qui utilise 2 threads et je grimpe deja a 50 % (si j'avais pas l'HT je serais a 100 %). Donc pour finir, mon conseil serai de baisser le nombre de thread et de les optimiser a fond.
@++

"Avant même de fonctionner, tout programme est déjà obsolète."

Quand je compare mon prog avec un autre genre scan100.exe( c'est un scanner 100 thread, il existe aussi en version 500 et 1000), ça me bouffe que dalle en ressource, j'atteind même pas les 2% du CPU.

Là je sèche!

La je t'avoue que je comprend pas non-plus...Un truc que j'avais pas remarquer au debut et que je ne sais pas si tu la modifier c'est la facon de fermer ton thread... tu utilise toujours CloseHandle(conf->hthread) ? parce que comme la dit BruNews, cela ne fait que cacher ton thread. A tu essayer terminatethread() ? Sinon j'imagine qu'en fait, meme la tache du thread termine, il doit rester en memoire .... ce qu'y fait qu'au final, 100 threads en memoire qui tourne pour rien .... c'est gore !!!
Je te conseil d'aller voir la msdn(je sais c'est lourd comme site mais c'est utile).
Tiens j'ai fait une petite recherhce vite fait. Va sur
=> [ Lien ]
En esperant que ca te soit utile.
@++

"Avant même de fonctionner, tout programme est déjà obsolète."

Jean84 allias Ion@ce
Webmaster de http://www.latriyade.com

Oui j'étais en train de voir ça!
Sinon j'ai essayé WaitForSingleObject
Ca marche, ça ne consomme presque plus rien, mais ça m'a méchement ralentit mon scanner!
On est sur la bonne voix!!!

for (;startip <= endip;startip++)
   {
                      while ( !nthread );
    {
    conf = (struct config *) malloc(sizeof(struct config) );
    conf->ip = startip;
    conf->port = port;
    conf->hthread = CreateThread(NULL, 0, &connection, conf, 0, NULL);
    WaitForSingleObject((conf->hthread),1000);
     nthread--;
    }
   }

Je connais pas du tout cette fonction. C'est quoi le but au juste ??? J'ai pas compris pour la fonction therminatethread()... t'as teste ou pas ??

"Avant même de fonctionner, tout programme est déjà obsolète."

Bon, j'ai test ça: TerminateThread(conf->hthread, 0); WaitForSingleObject((conf->hthread),200); ExitThread(0); D'après ce que j'ai compris, WaitForSingleObject attend (ici 200ms) avant de fermer le thread. TerminateThread et ExitThread n'ont rien changé à mon problème. WaitForSingleObject fonctionne mais le prog est ralenti!