[C] Allocazione struttura, problemi!

[Smoke666]

Ciao a tutti, in fase di esecuzione del mio programma, ricevo un errore di "segmentation fault", che presumo derivi dal fatto che non riesco ad allocare la memoria alla mia struttura. Posto un po di codice:

Strutture:

typedef struct scheda {
  /** puntatore lista degli autori */
  autore_t* autore;         
  /** titolo */
  char titolo[LLSTRING +1];   
  /** note */
  char nota[LLSTRING+1];    
  /** collocazione nella biblioteca */
  char collocazione[LSSTRING+1];  
} scheda_t;

typedef struct autore {
  /** cognome/i (almeno uno deve essere presente) */
  char cognome[LCOGN +1];
  /** nome/i (almeno uno deve essere presente) */ 
  char nome [LNOME + 1];
  /** puntatore al prossimo autore */ 
  struct autore * next; 
} autore_t;

Dunque, quando inizializzo una nuova scheda, faccio semplicemente una:

scheda_t * res=(scheda_t*)calloc(1, sizeof(scheda_t)); 
 res->autore = NULL;  /*è un puntatore, va inizializzato a NULL*/

Così, dovrei aver allocato lo spazio necessario per OGNI campo della struct, giusto? Probabilmente l'errore è qui, perchè quando uso questa scheda allocata sopra, mi da un segmentation fault...E' corretta l'allocazione? Grazie in anticipo!

[oregon]

Facci vedere in quale linea hai l'errore, oppure il codice che esegui ...

[Smoke666]

void main()
{scheda_t *res=NULL;
  res = new_scheda();
  if(res==NULL){printf("non allocata scheda\n");}
  else{printf("allocata! ora la libero...\n");
  		print_scheda(stdout, res); /*stampa la scheda vuota*/
  		free_scheda(&res); /*libera la scheda*/
  		if(res!=NULL){printf("non liberata\n");}
  		else{printf("liberata!\n");}
  	  }
  printf("fine\n");

}

void print_scheda (FILE* f, scheda_t * s)
{ 
char *strcognome;
          strcognome=malloc(sizeof(strlen(s->autore->cognome)+1));
          strcognome=strncpy(strcognome, s->autore->cognome, sizeof(strlen(s->autore->cognome)+1) );
          strcognome=strcat(strcognome, STRING_NULL);
}

Alla malloc di strcognome c'è il segmentation fault...penso sia dovuto al fatto che non allochi la scheda, quindi non può recuperare la lunghezza di s->autore->cognome...

[oregon]

Ma perché in

char* *strcognome;

due * ?

E se pensi sia l'allocazione, perché non mostri la funzione

new_scheda

?

[Smoke666]

Perdonami, il puntatore in più è un errore di battitura, la funzione di nuova scheda è quella che ho scritto sopra, la calloc e il puntatore autore a null...dovrebbe essere solo questa! In fondo, la calloc mi inizializza già i campi...

[oregon]

Ma la funzione restituisce anche un valore ...

Perché non la mostri tutta come richiesto?
In questo modo fai solo perdere tempo ...

[Smoke666]

scheda_t * new_scheda (void)   
{
 scheda_t * res=(scheda_t*)calloc(1, sizeof(scheda_t)); 
 res->autore = NULL; 
 if(res == NULL){return NULL;}
else	
return res; 
}

Non pensavo fosse significativo, scusami!

[oregon]

Originariamente inviato da Smoke666

scheda_t * new_scheda (void)   
{
 scheda_t * res=(scheda_t*)calloc(1, sizeof(scheda_t)); 
 res->autore = NULL; 
 if(res == NULL){return NULL;}
else	
return res; 
}

Non pensavo fosse significativo, scusami!

E' necessario avere tutto il quadro completo perché il problema potrebbe essere più di uno e in più parti del codice.

Il tuo è sicuramente in

strlen(s->autore->cognome)

dato che autore è NULL.

Molte parti del codice sono un po' "strane" ma non capisco proprio questa funzione

print_scheda

con il parametro FILE *f che non viene usato e in cui non si capisce cosa vorresti fare ...

[Smoke666]

Il parametro FILE *f serve per stampare la scheda (che sia vuota o piena), su un file, o sullo stdout. Quel "malloc(sizeof(strlen(s->autore->cognome)))" mi serve per copiare la stringa...Potrei però aggiungere un if, nel caso la scheda sia vuota anzichè fare quello, alloco una stringa di un solo char contentente il terminatore, e stampo quella...Potrebbe funzionare?

[oregon]

Ma perché devi copiare la stringa? Non si capisce ...