C:如何将malloc'ed指针传递到函数中并释放它们？

我正在使用套接字和线程开发一个客户机-服务器程序。我有一个客户端菜单，我正试图从客户端向服务器发送一个字符，然后调用相应的函数。我试图利用指针和malloc，但我不太明白如何传递它们并释放它们。我发现内存泄漏和错误，如：

/服务器“：双重免费或损坏fasttop:0x00007f37dc000a50***

及

分段故障

这是我的客户菜单：

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

do {

    // get a string from the server
    get_msg(sockfd);

    printf("\nFile and Information System\n");
    printf("===========================\n");
    printf("1. Show IP and Student ID\n");
    printf("2. Display server time\n");
    printf("3. Display system information\n");
    printf("4. List files on server\n");
    printf("5. File Transfer\n");
    printf("6. Exit\n");
    printf("Enter choice: ");
    scanf("%s", choice);

    //Send input to server
    send_menu_choice(sockfd, choice);

    switch (*choice) 
    {
        case '1':
                get_ip(sockfd);
            break;
        case '2':
                get_time(sockfd);
            break;
        case '3':
                get_and_send_uname(sockfd);
            break;
        case '4':
                get_file_list(sockfd);
            break;
        case '5':
                printf("File transfer not yet implemented\n");;
            break;
        case '6':
                printf("Exiting...\n");
            break;
        default:
                printf("Invalid choice\n");
    }

    //if(menu_choice != NULL)
    //{
    //    free(menu_choice);
    //}

} while (*choice != '6');

if (choice != NULL)
{
    free(choice);
}

将菜单选项发送到服务器：

void send_menu_choice(int socketNumber, char *choice)
{
    printf("Sending menu choice...\n");
    size_t n = strlen(choice) + 1;

    writen(socketNumber, (unsigned char *) &n, sizeof(size_t));
    writen(socketNumber, (unsigned char *) choice, n);

    printf("Sent choice: %s\n\n", choice);
}

服务器端：

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

do { 

    printf("Waiting for client to select option...\n\n");
    send_msg(connfd);

    get_menu_choice(connfd, menu_choice);
    printf("Client %d choice was: %s\n", connfd, menu_choice);

    switch (*menu_choice) 
    {
        case '1':
                send_ip(connfd);
            break;
        case '2':
                send_time(connfd);
            break;
        case '3':
                get_and_send_uname(connfd, *uts);
            break;
        case '4':
                send_file_list(connfd);
            break;
        case '5':
                printf("File Transfer not implemented\n");
            break;
        case '6': 
            break;
        default:
                printf("Invalid choice\n");
    }

    //if(menu_choice != NULL)
    //{
    //    free(menu_choice);
    //}

} while (*menu_choice != '6');

if (choice != NULL)
{
    free(choice);
}

void send_msg(int socket)
{
    char msg_string[] = "\nPlease enter an option:";

    size_t n = strlen(msg_string) + 1;
    writen(socket, (unsigned char *) &n, sizeof(size_t));   
    writen(socket, (unsigned char *) msg_string, n);
}

从客户端获取菜单选项：

void get_menu_choice(int socketNumber, char *choice)
{
    size_t n;

    readn(socketNumber, (unsigned char *) &n, sizeof(size_t));  
    readn(socketNumber, (unsigned char *) choice, n);

    printf("Received: %zu bytes\n\n", n);
}

所要求的其他职能：

发送消息服务器端：

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

do { 

    printf("Waiting for client to select option...\n\n");
    send_msg(connfd);

    get_menu_choice(connfd, menu_choice);
    printf("Client %d choice was: %s\n", connfd, menu_choice);

    switch (*menu_choice) 
    {
        case '1':
                send_ip(connfd);
            break;
        case '2':
                send_time(connfd);
            break;
        case '3':
                get_and_send_uname(connfd, *uts);
            break;
        case '4':
                send_file_list(connfd);
            break;
        case '5':
                printf("File Transfer not implemented\n");
            break;
        case '6': 
            break;
        default:
                printf("Invalid choice\n");
    }

    //if(menu_choice != NULL)
    //{
    //    free(menu_choice);
    //}

} while (*menu_choice != '6');

if (choice != NULL)
{
    free(choice);
}

void send_msg(int socket)
{
    char msg_string[] = "\nPlease enter an option:";

    size_t n = strlen(msg_string) + 1;
    writen(socket, (unsigned char *) &n, sizeof(size_t));   
    writen(socket, (unsigned char *) msg_string, n);
}

获取消息客户端：

void get_msg(int socket)
{
    char msg_string[32];
    size_t k;

    readn(socket, (unsigned char *) &k, sizeof(size_t));    
    readn(socket, (unsigned char *) msg_string, k);

    printf("%s\n", msg_string);
}

---

我做错了什么？我没有影响指针指向的位置的值吗？

请允许我将循环压缩到以下关键行：

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

do {

  scanf("%s", choice);
  switch (*choice) {
    case '6':
      printf("Exiting...\n");
      break;
    default:
      printf("Invalid choice\n");
  }

  if (choice != NULL)
  {
    free(choice);
  }

} while (*choice != '6');

这个循环有严重缺陷。您不仅使用1字节的内存来存储2个扫描，而且应该使用%c而不是%s，如果选择6以外的任何内容，循环不仅会访问已释放的内存，还会导致双重释放错误。另一方面，如果选择6，则在释放内存后访问内存，这也是一个非常严重的错误

循环终止后应释放内存。

在do循环中，重复释放指针

if(menu_choice != NULL)
{
    free(menu_choice);
}

但是，在循环之前只分配一次。

服务器端

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

分配一个字符

然后在名为get_menu_choiceconnfd的get_menu_choice中，使用1字节分配的menu_choice进行menu_choice

您将n个字节存储到选项中

这似乎不正确

---

一个字节的分配是否足够？意思是，n的取值是否大于1？

这是一个通用的答案，以下是我在项目中使用的函数：

/* this is safe free() and no not what you asked, check next one */
void my_free(void *ptr)
{
    if (ptr) {
        free(ptr);
    }
}

/* this is almost same as safe free above but also sets the pointer to NULL */
void my_freep(void **ptr)
{
    if (*ptr != NULL) {
        free(*ptr);
    }

    *ptr = NULL;
}

像这样称呼我的freep为我的freepvoid**&buffer； 注意，这里的缓冲区定义为指针，即char*buffer；。

---

希望这会有所帮助。

而且只分配一个字节供选择并不会给%s的scanf留下空间。哦，如果scanf无法工作，我该如何处理输入？我现在已经在循环后释放了内存，但在选择了几个选项后，我会被堆栈破坏。这一定是来自选项函数本身？所以我只是在解决堆栈崩溃问题，但这有帮助。感谢使用char*menu_choice=char*mallocsizeofchar；分配了足够的内存。。。printfSent选项：%s\n\n，选项；我可以做mallocsizeofchar+1；？或者像malloc8那样分配更多的空间；