Discussion :

[Tusbar]

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]

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

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]

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]

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

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

[Sve@r]

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

"(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]

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]

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