[C]Crash del programma con free()

[sam333]

ciao a tutti questo il codice del mio programmino...





#include <stdio.h>
#include <stdlib.h>
#include <string.h>


struct nomi{

char *n;

nomi *suc;

};
int main()
{

nomi *inizio,*lista;
char *a;
int i,n;

printf("Quanti nomi vuoi inserire?--> ");
scanf("%d",&n);
system("cls");
printf("Inseire prima stringa");
fflush(stdin);
a=(char*)malloc(sizeof(char));
gets(a);
inizio=(nomi*)malloc(sizeof(nomi));
inizio->n=(char*)malloc(strlen(a)*sizeof(char)+1);
strcpy(inizio->n,a);
lista=inizio;

for(i=1; i<n; i++){
system("cls");
a=(char*)malloc(sizeof(char));
lista->suc=(nomi*)malloc(sizeof(nomi));
lista=lista->suc;
printf("%d nome : ",i+1);
gets(a);
lista->n=(char*)malloc(strlen(a)*sizeof(char)+1);
strcpy(lista->n,a);
}
lista->suc=NULL;
fflush(stdin);


while(inizio!=NULL){

printf("\n%s",inizio->n);
inizio=inizio->suc;

}
system("pause");

free(inizio->n);
free(lista);
free(inizio);

system("pause");
free(a);
return 0;
}



come mai quando il programma arriva nel punto free(a); quindi dovrebbe liberare la memoria occupata da *a il programma crasha?
ho provato senza free(a); e il programma esce senza problemi ma io voglio liberare quella memoria...è strana come cosa ho sbaglio?

[Cait Sith]

se hai un crash sul free ci sono due possibilità:
- doppio free su quel puntatore
- accesso di memoria invalido su quel puntatore
nel primo caso lo risolvi assegnando NULL a tutti i puntatore liberati (se non succede è questo il motivo)
nel secondo caso bisogna trovare l'errore

se programmi su linux ti consiglio di usare valgrind,
valgrind [opzioni] tuo_programma
e lui se ci sono degli accessi alla memoria non validi te lo dice

[kwb]

Hmm, non so come hai fatto a compilare, perchè è sbagliato l'utilizzo della struttura nomi.
Se non definisci un nuovo tipo così:

typedef struct miastruttura{
int a;
char c;
}miastruttura;

Allora ogni volta che vuoi usare una variabile con questa struttura, devi dichiararla:

struct miastruttura miavariabile;

Prova ad aggiustare questa cosa e poi dicci.

[sottovento]

a=(char*)malloc(sizeof(char));
gets(a);

Qui allochi lo spazio necessario per un solo carattere, vale a dire il terminatore.
E' chiaramente insufficiente; di conseguenza, la stringa inserita andra' a "sporcare" memoria altrove. Alcuni compilatori inseriscono una "No man's land", ovvero alcuni byte con valori prefissati subito dopo la tua allocazione, appunto per verificare se ha sfondato.
Tali valori sono controllati al momento della free(). Cosi' vai in crash

[kwb]

Originariamente inviato da Antonio23

In C++ si puo' fare, evidentemente sta compilando in modalita' C++.

Ok, non lo sapevo!

Ma poi, qual è il significato di allocare un solo spazio per un carattere?

char *a;

a=(char*)malloc(sizeof(char));

Non era meglio fare un semplice char?

[sam333]

come potrei allocare più di un carattere?comunque sto usando visual c++ perchè è l'unico che mi va su windows 8..

[Vincenzo1968]

Originariamente inviato da sam333

come potrei allocare più di un carattere?comunque sto usando visual c++ perchè è l'unico che mi va su windows 8..

C'è anche GCC per Windows.

[sam333]

eh ok questo lo sapevo,ma come faccio a sapere quanti caratteri avrà la stringa che inserisce l'utente?mi servirebbe un modo per allocarla senza sapere il numero di caratteri..

[Vincenzo1968]

Originariamente inviato da sam333

eh ok questo lo sapevo,ma come faccio a sapere quanti caratteri avrà la stringa che inserisce l'utente?mi servirebbe un modo per allocarla senza sapere il numero di caratteri..

Ci vorrebbe la sfera di cristallo. Puoi allocare un buffer ragionevolmente grande(1024 byte) in modo da essere sicuro che riuscirà a contenere la stringa inserita dall'utente.

Dopo puoi calcolare la lunghezza della stringa inserita con strlen e riallocare il buffer con realloc.

[sam333]

mi puoi fare un esempio veloce veloce?..

[Vincenzo1968]

http://www.cplusplus.com/reference/cstdlib/realloc/

char *szMiaStringa = NULL;
int len;

szMiaStringa = (char*)malloc(sizeof(char) * 1024);
if ( !MiaStringa )
{
    printf("Impossibile allocare la memoria. Addio!\n");
    return -1;
}

printf("Inserisci una stringa: ");
fgets (szMiaStringa, 1024, stdin);

len = strlen(szMiaStringa);
szMiaStringa = realloc(szMiaStringa, len + 1);

if ( szMiaStringa )
    printf("Hai inserito la stringa \"%s\". La stringa inserita è lunga %d byte.\n", szMiaStringa, len);

/* Qui altro codice */

if ( szMiaStringa )
    free(MiaStringa);

[Vincenzo1968]

E comunque, se la memoria non è un problema, puoi evitare di riallocare il buffer:

char *szMiaStringa = NULL;
int len;

szMiaStringa = (char*)malloc(sizeof(char) * 1024);
if ( !MiaStringa )
{
    printf("Impossibile allocare la memoria. Addio!\n");
    return -1;
}

printf("Inserisci una stringa: ");
fgets (szMiaStringa, 1024, stdin);

len = strlen(szMiaStringa);

if ( szMiaStringa )
    printf("Hai inserito la stringa \"%s\". La stringa inserita è lunga %d byte.\n", szMiaStringa, len);

/* Qui altro codice */

if ( szMiaStringa )
    free(MiaStringa);

[Vincenzo1968]

Nientedimeno puoi evitare del tutto(sempre se lo spazio di memoria non è un problema) di allocare il buffer sullo heap:

char szMiaStringa[1024]
int len;

printf("Inserisci una stringa: ");

fgets (szMiaStringa, 1024, stdin);

len = strlen(szMiaStringa);

printf("Hai inserito la stringa \"%s\". La stringa inserita è lunga %d byte.\n", szMiaStringa, len);

/* Qui altro codice */

[sam333]

grazie provero ah ultimo cosa ho provato a compilare con Code::Blocks...ma come mai quando per esempio scrivo una frase con printf le parole me le segna di rosso?compilare compila ma da fastidio

[Vincenzo1968]

Originariamente inviato da sam333

grazie provero ah ultimo cosa ho provato a compilare con Code::Blocks...ma come mai quando per esempio scrivo una frase con printf le parole me le segna di rosso?compilare compila ma da fastidio

Perché, come ogni altro IDE(Visual Studio, etc), Code::Blocks evidenzia la sintassi. E' una caratteristica comoda che ti consente di distinguere al volo le varie parti sintattiche: identificatori, costanti, etc.

Se non ti piace il colore rosso puoi cambiarlo con quello che preferisci(o toglierlo del tutto).
Dovrebbe esserci la voce nel menù opzioni.



EDIT: Settings -> Editor -> Syntax highlighting -> String

[sam333]

nome ma me le segna di rosso sotto come se avessi scritto male,un po come fa word

[Vincenzo1968]

A me le segna in blu le stringhe:



Ma posso cambiare il colore attraverso "Settings -> Editor -> Syntax highlighting -> String".

Per esempio, se mi piace il rosso(e io sono di sinistra, quindi si, mi piace il rosso):

[sam333]

no a me mette dei segnetti sotto le parrole di errore come su word ,non capisco perchè...esempio printf("ciao a tutti");su ciao a tutti mi mette dei segnetti rossi

[Vincenzo1968]

Originariamente inviato da sam333

no a me mette dei segnetti sotto le parrole di errore come su word ,non capisco perchè...esempio printf("ciao a tutti");su ciao a tutti mi mette dei segnetti rossi

Allora non saprei. Puoi fare uno screenshot?

[sam333]

come?