[C]Aiuto per favore, vado incontro a continui crash su di un memcpy()

[Tubo Catodico]

Qualcuno sa perché questo codice crasha inevitabilmente sul memcpy()?

char *p_begin, *p_end, *app;
char begin[]={"<ParameterValue>"};

int i=0;


p_begin=strstr(input,begin);
p_begin=p_begin+16;
memcpy(app,p_begin,20);

Stringhe e puntatori a char non sono equivalenti in C...:wtf:

[gugoXX]

Devi allocare la memoria

[Tubo Catodico]

Per app? Già provato crasha uguale...

[gugoXX]

Puoi farci vedere un pezzo di codice completo (compreso dell'allocazione dello spazio destinazione), spiegandoci cosa avresti voluto fare, e che non funziona sulla memcpy?

[Tubo Catodico]

In pratica ho un file xml copiato pari pari in una stringa, senza spazi. Devo fare il parsing di alcuni valori contenuti all'interno e che stanno tra le due stringhe definitie da "begin" ed "end". Cerco le prime due occorrenze di queste stringhe, avanzo con p_begin di 16 caratteri (che è la lunghezza della stringa begin) e da lì devo copiare tutti i caratteri compresi tra questo punto e la stringa "end". Si pianta inevitabilmente sul memcpy()...

char *p_begin, *p_end, *app;
char begin[]={"<ParameterValue>"};
char end[]={"<\ParameterValue>"};



app=malloc(sizeof(char)*100);
p_begin=strstr(input,begin);
p_begin=p_begin+16;
p_end=strstr(p_begin,p_end);
memcpy(app,p_begin,(p_end - p_begin));

------------------------------------------

Trovato ho scambiato un end con un p_end............

[gugoXX]

Ti consiglio di non usare 16 come costante, ma di calcolarlo sulla base della prima stringa, usando strlen
Non usarei nemmeno 100 come costante, ma lo calcolerei sulla base della differenza di p_begin e p_end, una volta calcolati
Ti consiglio di non usare memcpy, dato che non aggiunge il blank finale. Userei strncpy, che copia al piu' n caratteri da una stringa sorgente ad una destinazione

[sottovento]

char *p_begin, *p_end, *app;
char begin[]={"<ParameterValue>"};
char end[]={"<\ParameterValue>"};

app=malloc(sizeof(char)*100);
p_begin=strstr(input,begin);
p_begin=p_begin+16;
p_end=strstr(p_begin,p_end);
memcpy(app,p_begin,(p_end - p_begin));



Questo codice e' pesantemente affetto da blind faith (fede cieca)
(controlla http://it.wikipedia.org/wiki/Fede_cieca_(informatica)):
1 - se la malloc() fallisce, vai in crash
2 - se la strstr() fallisce, vai in crash
3 - se la strstr() fallisce, vai in crash

Quale dei 3? Siccome dici che vai in crash sulla memcpy(), si puo' supporre che app sia NULL o p_begin sia NULL, oppure che p_end sia NULL

Controlla l'esito delle allocazioni o delle ricerche, eviti di perdere tempo per nulla

[Tubo Catodico]

Bè si, ancora manca la gestione degli errori, prima volevo solo farlo funzionare...
Comunque ho trovato l'errore, e come già detto, avevo stupidamente scritto un "p_end" (puntatore) al posto di un semplice "end" (stringa). Comunque grazie lo stesso di tutti i consigli, fanno sempre esperienza (questa cosa del blind faith per esempio mi giunge nuova) ;)

[cionci]

char *p_begin, *p_end, *app;
char begin[]={"<ParameterValue>"};
char end[]={"<\ParameterValue>"};



app=malloc(sizeof(char)*100);
p_begin=strstr(input,begin);
p_begin=p_begin+16;
p_end=strstr(p_begin,p_end);
memcpy(app,p_begin,(p_end - p_begin));

------------------------------------------

Trovato ho scambiato un end con un p_end............
app lo DEVI allocare...