Fuite memoire getpwuid

**Tusbar** · 28/02/2008, 04h38

Bonjour,

J'ai un petit problème que voici (UNIX) :

Je désire récupérer le chemin du dossier Home de l'utilisateur qui lance mon programme, pour ce j'utilise la fonction getpwuid, qui prend un uid_t en paramètre et retourne un pointeur sur struct passwd.

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
int             init_inf(t_data *d)
{
  struct passwd *p;
 
  d->uid = getuid();
  if ((p = getpwuid(d->uid)) != NULL)
    {
      if ((d->u_name = strdup(p->pw_name)) == NULL)
         return (FAILURE_E);
      if ((d->u_home = strdup(p->pw_dir)) == NULL)
         return (FAILURE_E);
      free(p->pw_name);
      return (SUCCESS_E);
    }
  show_error(2, PRGM ": " ENOINFO "\n");
  return (FAILURE_E);
}

Voici la fonction.
Il se trouve que j'ai quelques fuites de memoire :

in use at exit: 2301 bytes in 9 blocks.

J'ai essayé de free tous les pointeurs de la structure, il se trouve que seul pw_name ne me sort pas de warning.

(Les fuites ne proviennent pas des strdup)

Voici la structure :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 struct passwd {
                   char    *pw_name;       /* user name */
                   char    *pw_passwd;     /* encrypted password */
                   uid_t   pw_uid;         /* user uid */
                   gid_t   pw_gid;         /* user gid */
                   time_t  pw_change;      /* password change time */
                   char    *pw_class;      /* user access class */
                   char    *pw_gecos;      /* Honeywell login info */
                   char    *pw_dir;        /* home directory */
                   char    *pw_shell;      /* default shell */
                   time_t  pw_expire;      /* account expiration */
                   int     pw_fields;      /* internal: fields filled in */
           };

J'ai essayé avec getpwuid_r, mêmes fuites.

Voila, si quelqu'un connait une solution à ceci, c'est avec plaisir que je l'accepte.

Cdlt.

**Sve@r** · 28/02/2008, 08h41

Envoyé par Tusbar

Bonjour,

J'ai un petit problème que voici (UNIX) :

Je désire récupérer la le chemin du dossier Home de l'utilisateur qui lance mon programme, pour ce j'utilise la fonction getpwuid, qui prend un uid_t en paramètre et retourne un pointeur sur struct passwd.

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
int             init_inf(t_data *d)
{
  struct passwd *p;
 
  d->uid = getuid();
  if ((p = getpwuid(d->uid)) != NULL)
    {
      if ((d->u_name = strdup(p->pw_name)) == NULL)
         return (FAILURE_E);
      if ((d->u_home = strdup(p->pw_dir)) == NULL)
         return (FAILURE_E);
      free(p->pw_name);
      return (SUCCESS_E);
    }
  show_error(2, PRGM ": " ENOINFO "\n");
  return (FAILURE_E);
}

Voici la fonction.
Il se trouve que j'ai quelques fuites de memoire :

J'ai essayé de free tous les pointeurs de la structure, il se trouve que seul pw_name ne me sort pas de warning.

(Les fuites ne proviennent pas des strdup)

Hé si car d->u_name a été alloué par strdup() !!!
Donc faut que tu fasses un free(d->u_name) (et accessoirement aussi un free(d->u_home)) et non de p->pw_name (ni d'aucun autre membre de p car il n'y a pas eu d'allocation pour eux)...

Envoyé par Tusbar

Je désire récupérer la le chemin du dossier Home de l'utilisateur qui lance mon programme

Code :

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

1
2
3
4
5
 
char *home;
home=strdup(getenv("HOME"));
printf("%s\n", home);
free(home);

**Tusbar** · 28/02/2008, 09h00

Envoyé par Sve@r

Hé si car d->u_name a été alloué par strdup() !!!
Donc faut que tu fasses un free(d->u_name) (et accessoirement aussi un free(d->u_home)) et non de p->pw_name (ni d'aucun autre membre de p car il n'y a pas eu d'allocation pour eux)...

Code :

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

1
2
3
4
5
 
char *home;
home=strdup(getenv("HOME"));
printf("%s\n", home);
free(home);

"(Les fuites ne proviennent pas des strdup)"

Cela signifie que je les free plus tard dans mon programme, comme un millier d'autre peut être. Sans l'appel a getpwuid je n'ai aucune fuite dans mon programme.

Sinon, je n'ai pas demande de solution alternative, mais bien une aide par rapport a cette fonction. Si je n'utilise pas getenv, c'est qu'il ya une raison.

Cdlt.

EDIT :

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
#include <pwd.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/types.h>
 
int             main(void)
{
  struct passwd *p;
  char          *u_name;
  char          *u_home;
 
  if ((p = getpwuid(getuid())) != NULL)
    {
      u_name = strdup(p->pw_name);
      u_home = strdup(p->pw_dir);
 
      /* free(p->pw_name); */
 
      free(u_name);
      free(u_home);
      return (0);
    }
  return (1);
}

Valgrind :

==4351== malloc/free: in use at exit: 3325 bytes in 10 blocks.
==4351== malloc/free: 36 allocs, 26 frees, 37310 bytes allocated.

Sans les commentaires sur p->pw_name :

==4387== malloc/free: in use at exit: 2301 bytes in 9 blocks.
==4387== malloc/free: 36 allocs, 27 frees, 37310 bytes allocated.

Les memes fuites que plus haut...

**Médinoc** · 28/02/2008, 09h37

Que se passe-t-il si tu utilises getpwuid_r() à la place ?

**Tusbar** · 28/02/2008, 09h41

Exactement la meme chose, sauf que p->pw_name est deja free, en effet je lui passe l'adresse d'une structure en dur.

Cdlt.

**Sve@r** · 28/02/2008, 22h09

Envoyé par Tusbar

Sinon, je n'ai pas demande de solution alternative, mais bien une aide par rapport a cette fonction. Si je n'utilise pas getenv, c'est qu'il ya une raison.

Alors ne commence pas ton post par

je désire récupérer le HOME

mais plutôt par

j'ai un problème sur getpwuid

Envoyé par Tusbar

"(Les fuites ne proviennent pas des strdup)"

Cela signifie que je les free plus tard dans mon programme, comme un millier d'autre peut être. Sans l'appel a getpwuid je n'ai aucune fuite dans mon programme.

Alors faut tester avec un programme tout simple de ce style

Code :

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

1
2
3
4
5
6
int main()
{
  struct passwd *p;
  // p=getpwuid(getuid());
  show_error(2, PRGM ": " ENOINFO "\n");
}

Regarder si t'as une fuite mémoire, puis décommenter la 2° ligne et regarder de nouveau si t'as toujours ta fuite. Je l'aurais bien fait mais mon compilo Code::Block ne connait pas la fonction "show_error()"...

**Tusbar** · 29/02/2008, 01h46

Je me demande ou j'ai parle de récupérer une variable d'environnement.

(cf. le titre du thread)

Sve@r, regarde trois posts plus haut.

Les fuites de mémoire, merci je sais ce que je code, proviennent de getpwuid. Pas de show_error()... c'est un simple write(2) sur stderr.

La question n'est pas : "D'où proviennent mes fuites de mémoire ?"

Mais "Comment puis-je libérer la mémoire utilisée par getpwuid ?"

Merci, cordialement;

**souviron34** · 29/02/2008, 03h26

Envoyé par Tusbar

La question n'est pas : "D'où proviennent mes fuites de mémoire ?"

Mais "Comment puis-je libérer la mémoire utilisée par getpwuid ?"

tu as changé de question :

J'ai un petit problème que voici (UNIX) :

Je désire récupérer le chemin du dossier Home de l'utilisateur qui lance mon programme

Sve@r t'as donné la solution que n'importe quel programmeur de C ferait..

Si maintenant tu tiens absolument à utiliser getpwuid, n'insulte pas les autres en disant qu'ils n'ont rien compris...

Il y a peut-être un problème avec cette fonction, mais la manière normale d'obtenir le chemin complet du home d'un utilsateur est celle de Sve@r...(et en plus tu n'as même pas besoin d'utiliser strdup)...

NOTE : il y a 6 char * dans la structure, et tu n'en libères qu'un seul, le nom....

**Tusbar** · 29/02/2008, 03h45

Sinon faut lire les posts en entier ?

Je n'ai insulte personne.

Sve@r t'as donné la solution que n'importe quel programmeur de C ferait..

En effet, or il se trouve que quand j'exécute mon programme, l'environnement n'est pas encore défini.

NOTE : il y a 6 char * dans la structure, et tu n'en libères qu'un seul, le nom....

J'ai essayé de free tous les pointeurs de la structure, il se trouve que seul pw_name ne me sort pas de warning.

**souviron34** · 29/02/2008, 03h53

Envoyé par Tusbar

En effet, or il se trouve que quand j'exécute mon programme, l'environnement n'est pas encore défini.

euh......

Si tu as un utilsateur défini, tu as un environnement, non ??????????

**Tusbar** · 29/02/2008, 04h45

Pas si loin que ça...

J'ai un utilisateur défini, donc je crée l'environnement.

**Tusbar** · 29/02/2008, 18h07

Bon j'ai demande autour de moi...

Les fuites proviennent de la libc (pointeurs perdus).

Et bien on va se recoder un parser du fichier passwd hein.

Visiblement coder sans fuite de mémoire importe peu...

"Ce ne sont que 2301 bytes, t'as 2Go de RAM !!!" diront certains.

Merci pour vos réponses.

Fuite memoire getpwuid

C

Discussions similaires

Partager

Partager