Pb avec socket - too many open files

**be_tnt** · 17/05/2006, 08h24

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???

**gangsoleil** · 17/05/2006, 09h44

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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/

**be_tnt** · 17/05/2006, 10h07

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?

**fearyourself** · 17/05/2006, 10h10

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

**zul** · 17/05/2006, 10h13

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ... ).

**be_tnt** · 17/05/2006, 10h14

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.

**be_tnt** · 17/05/2006, 10h17

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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?

**Skyounet** · 17/05/2006, 10h17

Envoyé par be_tnt

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

Euh non c'est bien du C...

**be_tnt** · 17/05/2006, 10h19

Envoyé par Skyrunner

Euh non c'est bien du C...

bon j'avoue .... je suis nulle en C

**fearyourself** · 17/05/2006, 10h19

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Il faudrait pas utiliser close et non fclose?

Jc

**be_tnt** · 17/05/2006, 10h21

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

**fearyourself** · 17/05/2006, 10h24

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

**be_tnt** · 17/05/2006, 10h29

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
      }
   }
}

**be_tnt** · 17/05/2006, 10h31

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**fearyourself** · 17/05/2006, 10h35

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ca ne te paraît pas suspect comme code?

**be_tnt** · 17/05/2006, 10h36

Envoyé par fearyourself

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ca ne te paraît pas suspect comme code?

euh maintenant que tu le dis

**be_tnt** · 17/05/2006, 10h46

Alors j'ai modifié cette partie de code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
int port = strtol(ligne,NULL,10);
fclose(fb);
fb = NULL;
return port;

et en plus ça marche

Pb avec socket - too many open files

Réseau C

Discussions similaires

Partager

Partager