Segfault sur my_strcat()

Version imprimable

Bonjour,

Je suis en train de coder une fonction qui verifie le binaire dans chacun des paths
je voudrais concatener le path avec l executable et le passer sous access()
Quand j'utilise my_strcat , j ai "core dumped "
Voici un extrait du code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 void path(char *bin, char *path) { char **str; int i; i = 0; str = my_str_to_wordtab(path); while (str[i] != NULL) { str[i] = my_strcat(str[i], "/"); str[i] = my_strcat(str[i], binaire); printf("%s\n", str[i]); i++; } }
Si quelqu'un a une idee.
Cordialement,

08/02/2011, 15h44
jeroman
Citation:

Quand j'utilise my_strcat , j ai "core dumped "
(...)
Si quelqu'un a une idee.

Ben, sans boule de cristal, ça va être difficile...
Tu ne nous donnes pas le code de cette fonction, ni celui de my_str_to_wordtab (même si on imagine à quoi elles servent, on ne peut pas deviner comment elles sont codées).

En outre, si my_strcat a le même fonctionnement que strcat, pourquoi ne pas utiliser cette fonction standard ? et pourquoi écrire ce que j'ai mis en rouge ?
Code:

1 2 3 4 5 6 7 8 str = my_str_to_wordtab(path); while (str[ i ] != NULL) { str[ i ] = my_strcat(str[ i ], "/"); str[ i ] = my_strcat(str[ i ], binaire); printf("%s\n", str[ i ]); i++; }
si la fonction renvoie toujours la valeur du premier paramètre, c'est pas dangereux, mais inutile. Si elle peut renvoyer une autre valeur, la mémoire va fuiter.

Merci de ta reponse,, j ai essayer avec la fonction standart , j ai aussi un core dumped
J'ai ecris str[i] = ... pour recuperer le resultat dans cette variable et le passer ensute a access() mais il semble que je recupere des caracteres bizarres des fois ...

Le code de my_str_to_wordtab()

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
 
char    **my_str_wordtab(char *str)
{
  char  **tab;
  int   i;
  int   b;
  int   a;
 
  b = 0;
  i = 5;
  a = 0;
  if (str[i] == ':')
    while (str[i] == ':')
      i++;
  tab = malloc(sizeof(*tab) * (my_countword(str)));
  while (str[i] != '\n' && str[i] != '\0')
    {
      if (str[i] == ':' || str[i] == '\n')
        {
          while (str[i] == ':')
            i++;
          a = a + 1;
          b = 0;
        }
      tab[a] = malloc(sizeof(**tab) * (my_countchar(str + i)));
      while ((str[i] != ':')  && (str[i] != '\n') && (str[i] != '\0'))
        tab[a][b++] = str[i++];
      tab[a][b] = '\0';
    }
  tab[a + 1] = 0;
  return (tab);
}

Merci de ton aide .

08/02/2011, 19h07
jeroman

A priori, la mémoire réservée pour la concaténation n'est pas assez large.

Code:

tab[a] = malloc(sizeof(**tab) * (my_countchar(str + i)));

(Je suppose que my_countchar renvoie le nombre de caractères de la sous-chaine)
09/02/2011, 09h06
kimikimi

Bonjour,

Oui tu as raison, j ai fais quelque tests et je sefg sur strcat
Il ne me reste plus quá le refaire :)
Merci de ton aide
Cordialement,