Fuite memoire getpwuid

Version imprimable

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:

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 :

Citation:

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:

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.

Citation:
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:

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

Citation:

Envoyé par Tusbar

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

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

Citation:
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:

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:

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 :

Citation:

==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 :

Citation:

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

28/02/2008, 09h37
Médinoc

Que se passe-t-il si tu utilises getpwuid_r() à la place ?
28/02/2008, 09h41
Tusbar

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.
28/02/2008, 22h09
Sve@r
Citation:

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

Citation:

je désire récupérer le HOME

mais plutôt par

Citation:

j'ai un problème sur getpwuid

Citation:

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:

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()"...
29/02/2008, 01h46
Tusbar

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;
29/02/2008, 03h26
souviron34

Citation:

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

8O tu as changé de question :

Citation:

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

:aie:

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.... :aie:
29/02/2008, 03h45
Tusbar

Sinon faut lire les posts en entier ? :roll:

Je n'ai insulte personne.

Citation:

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.

Citation:

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

Citation:

J'ai essayé de free tous les pointeurs de la structure, il se trouve que seul pw_name ne me sort pas de warning.
29/02/2008, 03h53
souviron34

Citation:

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 ??????????
29/02/2008, 04h45
Tusbar

Pas si loin que ça...

J'ai un utilisateur défini, donc je crée l'environnement.
29/02/2008, 18h07
Tusbar

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. :roll:

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.