Pb avec socket - too many open files

Version imprimable

17/05/2006, 08h24
be_tnt

Pb avec socket - too many open files

Bonjour!

De si bon matin, j'attaque dur. J'ai développé une librarie en C. Je ne m'y connais pas beacoup sur ce point. Actuellement cette library contient 2-3 fonctions appelés par un autre programme (que je ne contrôle pas). En fait cette librarie est utilisé pour rediriger une demande d'accès vers un daemon écrit en Java. Le tout est basé sur UDP. Mon problème est qu'à chaque tentative de connection ma librairie crée un nouveau datagram socket. Résultat: si un user s'amuse à lancer en bloque des tentatives de connection, un moment donné, j'obtiens le message "Too many open files" et donc je ne sais plus créer de nouvelle socket. Maintenant ce que je cherche à faire est de ne créer qu'un seul socket et de le ré-utiliser. Est-ce possible? Si oui, comment je peux faire? En Java, j'utiliserai le constructeur de classe pour initialiser mon objet une seule fois. mais en C???
17/05/2006, 09h44
gangsoleil

Bonjour,

Je ne suis pas bien sur de comprendre exactement ce qui te pose problème : lorsque tu ouvres une socket, tu récupères son identifiant. Ensuite, pour écrire sur cette socket, tu écris sur cet identifiant, dans le genre de ceci :

Code:

send(identifiant_de_socket, "Hello world!\n", .........);

Un bon début sur les sockets : http://c.developpez.com/WalrusSock/
Une bonne suite, en anglais : http://beej.us/guide/bgnet/
17/05/2006, 10h07
be_tnt

huh il sembelerait que le problème vient d'un fichier que j'ouvre et qu'apparemment le fd ne se ferme pas correctement. Avec la commande lsof, je vois ce fichier ouvrt un nombre impressionnant de fois :s

Ce que je comprends pas est que je fais fclose. J'ai même ajouté juste après fd = NULL. Mais ça ne change rien :?

Comme il s'agit d'une library, est-ce qu'il y a d'autres choses à faire?
17/05/2006, 10h10
fearyourself

Citation:

Envoyé par be_tnt

J'ai développé une librarie en C.

On dit bibliothéque.

Pourquoi ne pas utiliser close?

C'est soit à toi de faire une fonction qui referme les sockets, de façon interne ou alors que l'utilisateur appelle une fonction qui le fera.

Soit tu laisses l'utiliser fermer la socket avant de rappeler tes fonctions...

Jc
17/05/2006, 10h13
zul
Pourquoi ne fais tu pas exactement la même chose en C ? Evidemment tout n'est pas automatique en C mais les bonnes pratiques de programmation peuvent rester.
Code:

1 2 3 4 5 6 7 #define INVALID_SOCKET -1 static int my_socket = INVALID_SOCKET; if ( my_socket == INVALID_SOCKET ) my_socket = socket( ... );
Que tu peux encapsuler dans une fonction init_socket ( si tu veux faire d'auter traitement ... ).
17/05/2006, 10h14
be_tnt

humm désolé si l'anglais prend le pas sur le français parfois ....

Comme dit dans mon précédent message, le pb semble venir d'un FD qui ne serait pas correctement libéré mais je fais déjà fclose(fd) et même fd = NULL.
17/05/2006, 10h17
be_tnt
Citation:
Envoyé par zul

Pourquoi ne fais tu pas exactement la même chose en C ? Evidemment tout n'est pas automatique en C mais les bonnes pratiques de programmation peuvent rester.

Code:

1 2 3 4 5 6 7 #define INVALID_SOCKET -1 static int my_socket = INVALID_SOCKET; if ( my_socket == INVALID_SOCKET ) my_socket = socket( ... );

Que tu peux encapsuler dans une fonction init_socket ( si tu veux faire d'auter traitement ... ).
euh je suppose que là c'est plus du C++, non?
17/05/2006, 10h17
Skyounet

Citation:

Envoyé par be_tnt

euh je suppose que là c'est plus du C++, non?

Euh non c'est bien du C...
17/05/2006, 10h19
be_tnt

Citation:

Envoyé par Skyrunner

Euh non c'est bien du C...

bon j'avoue .... je suis nulle en C :aie:
17/05/2006, 10h19
fearyourself

Citation:

Envoyé par be_tnt

huh il sembelerait que le problème vient d'un fichier que j'ouvre et qu'apparemment le fd ne se ferme pas correctement. Avec la commande lsof, je vois ce fichier ouvrt un nombre impressionnant de fois :s

Ce que je comprends pas est que je fais fclose. J'ai même ajouté juste après fd = NULL. Mais ça ne change rien :?

Comme il s'agit d'une library, est-ce qu'il y a d'autres choses à faire?

Euhhh depuis quand des sockets sont des pointeurs?

Code:

int socket(int domain, int type, int protocol);

Il faudrait pas utiliser close et non fclose?

Jc
17/05/2006, 10h21
be_tnt

euh en fait là je parlais de fichier réellement et pas de socket ....
17/05/2006, 10h24
fearyourself

Citation:

Envoyé par be_tnt

euh en fait là je parlais de fichier réellement et pas de socket ....

Bon il serait peut-être temps de demander un code réduit au minimum, compilable qui montre le problème?

Jc

alors voici la partie du code qui ouvre le fichier, extrait l'info pis ferme ce fichier:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
/*
 * Retrieve the UDP port to which requests
 * must be sent.
 * Return the port number or -1 if failed
*/
int getPort(char* path) {
   FILE *fb;
   char ligne[20];
   char filename[50];
 
   /* Build the filename */
   strncpy(filename,path,sizeof(filename)-1);
   strncat(filename,"/etc/.authServerPort",sizeof(filename)-1-strlen(path));
 
   /* Retrieve the UDP port to which send the authentication request */
   if ( (fb = fopen(filename,"r")) == NULL) {
      /* Log the error message */
      if ( DEBUG == 1 ) {
         fprintf(logfile , "ERROR: Can not open port file: %s\n", strerror(errno));
         fflush(logfile);
         fclose(logfile);
      }
      return -1;
   } else {
      /* read the UDP port */
      if (fgets(ligne,sizeof(ligne),fb) != NULL) {
         return strtol(ligne,NULL,10);
         fclose(fb);
         fb = NULL;
      } else {
         /* Log the error message */
         if (DEBUG == 1) {
            fprintf(logfile , "ERROR: Can not read port file: %s\n", strerror(errno));
            fflush(logfile);
            fclose(logfile);
         }
         fclose(fb);
         fb = NULL;
         return -1;
      }
   }
}

La commande lsof montre que .authServerPort est toujours "ouvert"
Code:

1 2 qip-login 15965 root 254r VREG 32,4 5 3994687 /opt/qip/etc/.authServerPort

17/05/2006, 10h35
fearyourself
Code:

1 2 3 return strtol(ligne,NULL,10); fclose(fb); fb = NULL;
Ca ne te paraît pas suspect comme code? :wink:
17/05/2006, 10h36
be_tnt
Citation:
Envoyé par fearyourself

Code:

1 2 3 return strtol(ligne,NULL,10); fclose(fb); fb = NULL;

Ca ne te paraît pas suspect comme code? :wink:
euh maintenant que tu le dis :aie:
17/05/2006, 10h46
be_tnt
Alors j'ai modifié cette partie de code:
Code:

1 2 3 4 5 int port = strtol(ligne,NULL,10); fclose(fb); fb = NULL; return port;
et en plus ça marche :merci: