C выделения памяти в структуре студентов не работает

Question

я должен выполнять эти функции в internal.h ниже:

#ifndef STUDENT_INTERNAL_H 
#define STUDENT_INTERNAL_H 

#include "student.h" 

/** Allocate memory for a new @ref student object. */ 
struct student *student_alloc(void); 

/*** Initialize an already-allocated @ref student object. 
* @returns NULL on failure but does not free the memory 
*/ 

struct student *student_init(struct student *, const char *name, size_t namelen, 
          struct student_id, student_complete); 

#endif /* STUDENT_INTERNAL_H */ 

Это то, что я до сих пор, но я смущен, и это не работает:

#include <sys/types.h> 
#include "common.h" 
#include "student.h" 
#include "transcript.h" 
#include "internal.h" 

/** 
* Initialize an already-allocated @ref student object. 
* @returns NULL on failure but does not free the memory 
*/ 
struct student *student_init(struct student *student, const char *name, size_t namelen, 
          struct student_id stud_id, student_complete stud_complete) { 

    struct student *initialized_student; 

    name = malloc(sizeof(namelen)); 
    initialized_student = malloc(sizeof (student)); 

    if (student_alloc()) { 
     initialized_student->s_name = name; 
     initialized_student->s_id = stud_id; 
     initialized_student->s_complete = stud_complete; 

     return initialized_student; 
    } else { 
     return 0; 
    } 
} 

stud_complete здесь указатель функции, которая объявлена ​​в заголовочном файле отводящий student.h как typedef int (*student_complete)(struct student *);

Answer 1

API запутан:

Что такое namelen?

• Это длина исходного префикса name, который является именем студента? В этом случае вы должны выделить один дополнительный байт для нулевого терминатора и скопировать имя соответствующим образом.
• Это максимальная длина имени студента? Опять же, если это длина, а не размер, вам нужен дополнительный байт для нулевого терминатора.
• в любом случае sizeof(namelen) не подходит.

Что делает student_alloc()?

Аналогично, размер передается malloc() выделить student структуру должно быть таким struct student или более надежно, что такого типа, на который указывает указатель назначения.

Вот исправленный вариант:

struct student *student_init(struct student *student, const char *name, size_t namelen, 
          struct student_id stud_id, student_complete stud_complete) { 

    struct student *initialized_student = malloc(sizeof(*initialized_student)); 
    char *s_name = calloc(namelen + 1); 

    if (initialized_student && s_name && student_alloc()) { 
     strncat(s_name, name, namelen); 
     initialized_student->s_name = s_name; 
     initialized_student->s_id = stud_id; 
     initialized_student->s_complete = stud_complete; 
     return initialized_student; 
    } else { 
     free(initialized_student); 
     free(s_name); 
     return 0; 
    } 
}