[C] Lista semplice/Segmentation Fault

[C3ls1us]

Vi riassumo il mio problema:
stò provando ad implementare in C delle liste semplici e a prendere dimestichezza con i puntatori a struct.
Il mio esercizio consisteva nell'implementare una lista semplice, inserire dei dati, e poter visualizzarli in seguito.
Probabilmente si tratta di un errore sintattico, ma è da un giorno che sono fermo, nonostante abbia sfogliato kg di pagine web e non...! Grazie Mille!!!

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

typedef struct lista
{
  int x;
  int y;
  struct lista *succ;
}ls;

struct lista *pt,*punt;  //pt e pt ausiliario

int main()
{
  int n;

  printf("Num di elementi\t");
  scanf("%d",&n);
  
  pt=NULL;  //inizializzazione
  pt=(ls*)malloc(sizeof(ls)*2);
  if (pt==NULL)
    return 1;  //controllo su malloc
  punt=pt;

  printf("ins dati\n"); 
  while(n>0)
  {
    scanf("%d",&pt->x);
    scanf("%d",&pt->y);
    n--;
    pt=pt->succ;
  }
  pt=punt;
  printf("%d,%d\n",pt->x,pt->y);
  free(pt);
  return 0;
}

mattia@mattia:~/c$ ./a.out
Num di elementi 3
ins dati
1
1
2
Segmentation fault

[k0nt3]

beh funziona solo se inserisci un solo elemento, perchè allochi memoria solo per il primo elemento. il fatto che moltiplichi size per 2 non ha effetto in quanto il prossimo elemento non sarà necessariamente in un indirizzo di memoria sequenziale al primo elemento (altrimenti sarebbe un array e non una lista).
ecco una versione funzionante

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

typedef struct lista
{
  int x;
  int y;
  struct lista *succ;
}ls;

ls *testa = NULL;   //  testa della lista
ls *cur = NULL;  //  cursore

int main()
{
  int n;

  printf("Num di elementi\t");
  scanf("%d",&n);

  printf("ins dati\n"); 
  while(n>0)
  {
    ls *elem = (ls*)malloc(sizeof(ls));
    scanf("%d",&elem->x);
    scanf("%d",&elem->y);
    
    if(cur==NULL) // la lista e' vuota
    {
      cur = elem;  // il cursore punta all'elemento che ho appena creato
      testa = cur;    // il nuovo elemento diventa anche la testa della lista
    }
    else  // altrimenti cur punta all'ultimo elemento inserito nella lista
    {
      cur->succ = elem;  // il campo succ dell'ultimo elemento inserito punta al nuovo elemento creato
      cur=cur->succ;    // si scorre il cursore all'ultimo elemento creato
    }

    --n;
  }

  // parto dal primo elemento per liberare la memoria
  cur = testa;
  while(cur != NULL)
  {
    printf("%d,%d\n",cur->x,cur->y);
    free(cur);
    cur = cur->succ;  // quando arrivo alla fine della lista cur puntera' a NULL interrompendo il while
  }

  return 0;
}

come vedi alloco memoria ogni volta che inserisco un elemento

[C3ls1us]

Grazie!!!!! Mi sfuggiva qualcosa! Grazie davvero!