[C] Realloc() non si comporta correttamente all'interno di una funzione

[magicofred]

ciao a tutti, volevo capire come mai mettendo del codice in una funzione e richiamando quella funzione non va, mi da un errore di realloc, mentre lasciandolo nel main è tutto perfettamente fuznionante.. vi copio il codice

int p[2];

pipe(p);

int isfork;
isfork = fork();

if(isfork == 0){

dup2(p[1],1);
close(p[0]);
  char *args[2] = { "date", NULL };
execvp (args[0], args);
}

else
{

char c;
char *stringa = malloc(sizeof(char *));
int i = 0;

dup2(p[0],0);
close(p[1]);
while( read(p[0],&c,sizeof(c) ) ){
stringa = realloc(stringa, (i + 1) * sizeof(char *));
i++;
strcat(stringa,&c);
printf("%c",c);
}

printf("%s",stringa);
}

se questo lo metto in una funzione void, o int , o char che sia mi da il seguente errore

*** glibc detected *** realloc(): invalid next size: 0x0804a008 ***
SAborted

non reisco proprio a capire l'errore qual è se funziona nel main.. boh.. ditemi voi.. grazie.

[oregon]

Ma perche' allochi puntatori a char invece di char ?

Non deve essere

char *stringa = malloc(sizeof(char *));
e
stringa = realloc(stringa, (i + 1) * sizeof(char *));

ma

char *stringa = malloc(sizeof(char));
e
stringa = realloc(stringa, (i + 1) * sizeof(char));

P.S. Usa un titolo piu' significativo ...

[magicofred]

ho provato a usare puntatori char invece di puntatori a char, ma il risultato nn cambia
ho cambiato il nome.. grazie cmq

[oregon]

Originariamente inviato da magicofred
ho provato a usare puntatori char invece di puntatori a char, ma il risultato nn cambia
ho cambiato il nome.. grazie cmq

Che vuoi dire? Puntatori char invece che puntatori a char?

E poi non puoi usare la strcat

strcat(stringa,&c);

con un dato di tipo char ... devono essere due puntatori a char, terminati da un carattere 0 binario, cioe' una stringa.

Usando la strcat in questo modo, hai un funzionamento anomalo non prevedibile, dipendente dalla situazione della memoria in quel momento, cosa che giustificherebbe il comportamento diverso in diverse situazioni.

In ogni caso, stai usando in modo sbagliato i puntatori a char e le funzioni associate ...

[magicofred]

Originariamente inviato da oregon
Che vuoi dire? Puntatori char invece che puntatori a char?

E poi non puoi usare la strcat

strcat(stringa,&c);

con un dato di tipo char ... devono essere due puntatori a char, terminati da un carattere 0 binario, cioe' una stringa.

Usando la strcat in questo modo, hai un funzionamento anomalo non prevedibile, dipendente dalla situazione della memoria in quel momento, cosa che giustificherebbe il comportamento diverso in diverse situazioni.

In ogni caso, stai usando in modo sbagliato i puntatori a char e le funzioni associate ...

mi suggereresti una soluzione o mi diresti in cosa sbaglio? magari con un esempio, grazie

[oregon]

Mah ... non lo posso provare, ma scriverei qualcosa del genere ...

char c;
char *stringa = malloc(sizeof(char));
stringa[0]=0;
int i = 0;

dup2(p[0],0);
close(p[1]);
while( read(p[0],&c,sizeof(c) ) )
{
  i++;
  stringa = realloc(stringa, (i + 1) * sizeof(char));
  stringa[i-1]=c;
  stringa[i]=0;
}

printf("%s",stringa);

[magicofred]

Originariamente inviato da oregon
Mah ... non lo posso provare, ma scriverei qualcosa del genere ...

char c;
char *stringa = malloc(sizeof(char));
stringa[0]=0;
int i = 0;

dup2(p[0],0);
close(p[1]);
while( read(p[0],&c,sizeof(c) ) )
{
  i++;
  stringa = realloc(stringa, (i + 1) * sizeof(char));
  stringa[i-1]=c;
  stringa[i]=0;
}

printf("%s",stringa);

GRAZIE 100000