C: Segmentation Fault 11 strcpy 指针数组

Question

The code compiles with no errors, but I get a segmentation Fault Error (11), I'm guessing the problem is when I use the strcpy() function in line 82 ,something goes wrong, why im not getting any compile errors and how can I fix it, are the char array pointers well implemented?. I can not modiffy the current structs of the code.

#include "stdio.h"
#include "stdlib.h"
#include "string.h"

typedef struct
{
  int n_uea;
  char **nombre;
  double *calificacion;
} UEA;

typedef struct
{
  int n_pais;
  char *nombre[50];
} PAIS;

typedef struct
{
  char nombre[50];
  UEA uea;
  PAIS *pais;
} ALUMNO;

void AllocPais(PAIS *p, int np){
  p = (PAIS*) malloc(sizeof(PAIS));
  p = &p[0];
  p->n_pais = np;
  for (int i = 0; i < np; i++) {
    p->nombre[i] = (char*) malloc(sizeof(char)*50);
  }
}
void AllocUEA(UEA *u , int nu){
  u->n_uea = nu;
  u->nombre = (char **) malloc(sizeof(char*)*nu);
  for (int i = 0; i < nu; i++) {
    u->nombre[i] =(char*) malloc(sizeof(char)*50);
  }
  u->calificacion = (double*) malloc(sizeof(double) * nu);
}
void AllocAlumnoMemory(ALUMNO *a, int np, int nu){
  AllocPais(a->pais,np);
  AllocUEA(&(a->uea),nu);
}

int main(int argc, char const *argv[]) {
  ALUMNO* arreglo;
  int entradas;
  printf("%s\n","Ingrese el numero de Entradas");
  scanf("%d",&entradas);
  arreglo = malloc(sizeof(ALUMNO)*entradas);
  for (int i = 0; i < entradas; i++) {
    char separator[10];
    char name[20];
    int numUea , numPaises;
    printf("%s\n","Se espera separador");
    scanf("%s",separator);
    printf("%s\n","Ingrese el nombre");
    scanf("%s",name);
    strcpy(arreglo[i].nombre,name);


    printf("%s\n","Ingrese el numero de UEA");
    scanf("%d",&numUea);
    AllocUEA(&arreglo[i].uea,numUea);
    for (int a = 0; a < numUea; a++) {
      char name [15];
      double cal;
      scanf("%s %lf", name, &cal);
      strcpy(arreglo[i].uea.nombre[a],name);
      arreglo[i].uea.calificacion[a] = cal;
    }

    printf("%s\n","Ingrese Numero de paises");
    scanf("%d",&numPaises);
    PAIS nuvp;
    arreglo[i].pais = &nuvp;
    AllocPais(arreglo[i].pais,numPaises);
    for (int b = 0; b < numPaises; b++) {
      char names [15];
      scanf("%s",names);
      strcpy(arreglo[i].pais->nombre[b],names);
    }

  }


  return 0;
}

Answer 1

Please note down below points and try them:

1. Avoid using strcpy/strcat etc. they do not protect against buffer overruns.

2. Use strlcpy/strlcat instead.

3. When using strncpy , make sure to NULL terminate the string buffer.