strtok causa seg fault

[Joker91]

come da titolo, la strtok mi causa seg fault... Con ddd ho notato che mi va in seg fault subito alla prima chiamata... vi copio il codice togliendo le parti non attinenti:

// char* r è una stringa passata dalla funzione

char* p= r;
char* tok;

tok = strtok(p, ";");
	
while (tok != NULL){
....
....
....
....
   tok = strtok(NULL, ";");
}

Cosa c'è di sbagliato? :| Se volete il resto del codice posso postarlo ma è solo la parte operativa della funzione che lavora con quello che tira fuori strtok

[lorenzo001]

Sicuramente perché hai usato una stringa costante passata con char* r

[Joker91]

Quindi dici che dovrei copiare r in un array dinamico?

[pabloski]

Originariamente inviato da Joker91

Quindi dici che dovrei copiare r in un array dinamico?

no, lui si riferiva al fatto che quando hai chiamato quella funzione, hai usato una cosa tipo func("Ciao mondo");

invece devi dichiarare la stringa come array di char, char a[] = "Ciao mondo"; e poi func(a);

[lorenzo001]

Originariamente inviato da pabloski

no, lui si riferiva al fatto che quando hai chiamato quella funzione, hai usato una cosa tipo func("Ciao mondo");

invece devi dichiarare la stringa come array di char, char a[] = "Ciao mondo"; e poi func(a);

Esatto

[Joker91]

r è una stringa chiamata proprio in quel modo eh... char r[] = "......."

[lorenzo001]

Allora mostra *tutto* il codice che serve per determinare il problema, altrimenti si va solo a "ipotesi".

Ovvero, posta il codice minimo compilabile con cui riprodurre l'errore.

[Joker91]

char * record[]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

int main (void) {
int i;

for (i=0; record[i]!=NULL; i++) {
func(record[i]);
}

return 0;
}

func(char * r){

char* p= r;
char* tok;

tok = strtok(p, ";");

while (tok != NULL){
....
....
....
....
tok = strtok(NULL, ";");
}

[lorenzo001]

Originariamente inviato da Joker91

char * record[]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

Avevi detto che r è una stringa chiamata proprio in quel modo eh...

char r[] = "......."

ma non mi pare sia così dato che non hai usato una stringa ma un array di puntatori a stringhe costanti

char * record[]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

Essendo comunque delle stringhe costanti avrai quel problema in quanto il loro contenuto non può essere modificato come tenta di fare la strtok.

Quindi, o usi nel tuo codice dello spazio allocato dinamicamente in memoria in cui passi le varie stringhe, o dichiari le stringhe così

char p1[] = "BLA;BLA";
char p2[] = "AS;DA;SD";
char p3[] = "JISA;DJIA;SD";
char *record[]= {  p1, p2, p3, NULL };

[pabloski]

Quoto lorenzo e aggiungo che ciò è dovuto al fatto che lì stai inizializzando un array di puntatori, ma in nessun modo stai stabilendo le caratteristiche delle singole stringhe ( in particolare la dimensione e quindi la presenza o meno del terminatore nullo ).

Una sintassi simile alla tua è questa

char record[][128]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

128 è ovviamente la dimensione massima possibile per le singole stringhe

[lorenzo001]

Originariamente inviato da pabloski

Una sintassi simile alla tua è questa

char record[][128]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

Attenzione però perché, in questo caso, il codice del ciclo va modificato così

for (i=0; record[i][0]!=NULL; i++)

[pabloski]

Giusto.

Potrebbe pure usare una stringa vuota come ultimo elemento ed utilizzare strcmp o strlen.

[Joker91]

uhm ho un lapsus... l'array modificabile è dichiarato così

char r[]= "..."

e quello non modificabile con il puntatore? Non c'entra la memoria heap o stack?

[pabloski]

Originariamente inviato da Joker91

uhm ho un lapsus... l'array modificabile è dichiarato così

char r[]= "..."

questo qui sarebbe il parametro di func? ma in ogni caso i due modi sono equivalenti

Originariamente inviato da Joker91

e quello non modificabile con il puntatore? Non c'entra la memoria heap o stack?

no, quella è una variabile globale e quindi statica e viene piazzata nella sezione data dell'eseguibile, non allocata in heap o stack

in heap ci vanno tutti gli oggetti allocati con malloc

nello stack ci vanno invece le variabili automatiche ( locali ), cioè i parametri di una funzione, le variabili definite all'interno di una funzione ( senza static ovviamente )

[Joker91]

uhm ok, quindi qual'è la sintassi per l'array modificabile e non?

[pabloski]

in che senso modificabile? il problema non era risolvere il segmentation fault?

lo risolvi o facendo come ti ha indicato lorenzo e cioè

char p1[] = "BLA;BLA";
char p2[] = "AS;DA;SD";
char p3[] = "JISA;DJIA;SD";
char *record[]= {  p1, p2, p3, NULL };

o come ho detto io

char record[][128]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

stando però attento a modificare il for

for (i=0; record[i][0]!=NULL; i++)

[Joker91]

A quanto avevo capito il problema era sul fatto che la stringa non è modificabile...

In ogni caso io da specifica non posso modificare il char* record... Quindi la cosa migliore è copiarmi quelle stringhe in altri array no? E questi array li dichiaro normalmente?

[pabloski]

Originariamente inviato da Joker91

A quanto avevo capito il problema era sul fatto che la stringa non è modificabile...

In ogni caso io da specifica non posso modificare il char* record... Quindi la cosa migliore è copiarmi quelle stringhe in altri array no? E questi array li dichiaro normalmente?

ah! e chi diavolo l'ha stabilita quella specifica? se ti hanno chiesto di creare un'array di stringhe in quel modo, beh, ne capiscono poco di C

comunque se non puoi cambiare quella definizione, allora devi assolutamente fare come dice lorenzo

[lorenzo001]

Se non vuoi/puoi assolutamente toccare la linea

char * record[]= { "BLA ;BLA" , "AS;DA;SD" , "JISA;DJIA;SD" , NULL };

allora dovrai prima copiare le varie stringhe costanti in vettori allocati in memoria e passarli alla funzione.

Oppure - come ulteriore alternativa - non usare la funzione strtok ma scrivi una tua funzione che faccia quello che fa la strtok ma senza scrivere nella stringa sorgente.

[Joker91]

ok risolto questo problema della strtok ora ne ho un altro... Faccio un esempio:

ho la stringa p = "culo; tette"

faccio tok = strtok(p, ";") e tok diventa "culo"...

Quindi a questo punto c'è il ciclo while (tok != NULL)...
Poi alla fine del ciclo c'è tok = strtok(NULL, ";")... E a quel punto è come se mi facesse la strtotk sulla tok e quest'ultima mi diventa NULL... E quindi finisce già il ciclo... Ma dov'è finito il resto della stringa p?

Come mai mi succede questo? Ho preso il ciclo pari pari da qui: http://www.cplusplus.com/reference/c...string/strtok/


EDIT: ah mi son dimenticato di dire che ho fatto in modo che tok non venga mai toccata se non da strtok