C - 指向结构指针的指针 - 分段错误

Posted 2021-04-16

我学习C指针，因此我尝试使用指针表示法创建结构数组。我分配了指向struct User的指针数组，我设置名称和标识符，然后我尝试打印它然后崩溃。

Dockerfile：

FROM    ubuntu:xenial

RUN     apt-get update 
            && apt-get install -y 
                software-properties-common 
                wget 
            && add-apt-repository -y ppa:ubuntu-toolchain-r/test 
            && apt-get update 
            && apt-get install -y 
                make 
                git 
                curl 
                vim 
                vim-gnome 
            && apt-get install -y cmake=3.5.1-1ubuntu3 
            && apt-get install -y 
                gcc-4.9 g++-4.9 gcc-4.9-base 
                gcc-4.8 g++-4.8 gcc-4.8-base 
                gcc-4.7 g++-4.7 gcc-4.7-base 
                gcc-4.6 g++-4.6 gcc-4.6-base 
            && update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-4.9 100 
            && update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-4.9 100


COPY ./*.c /tmp/

WORKDIR /tmp/
RUN gcc -Wall main.c -o main

ENTRYPOINT ./main

我在docker容器中运行我的程序，因为我想要有未使用的内存。

我的节目：

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

#define MAX_USERS_NUMBER 10

void set_users();
void print_users();

typedef struct User
{
    int id;
    char* username;
} User;

User **users;

int main(int argc, const char* argv[])
{
    set_users();
    print_users();
    return 0;
}

void set_users()
{
    users = (User **) malloc(MAX_USERS_NUMBER * sizeof(User *));
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        *(users + i) = (User *) malloc(sizeof(User));
        (*users + i)->id = i;
        (*users + i)->username = (char *) malloc(strlen("username") + 1);
        strcpy((*users + i)->username, "username");
        printf("%d. id: %d, username: %s
", i, (*users + i)->id, (*users + i)->username);
    }
}

void print_users()
{
    printf("
Users
");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s
", i, (*users + i)->id, (*users + i)->username);
    }
}

我按命令构建图像：

docker build -t gcc-poc .

我按命令运行它：

docker run -it gcc-poc /bin/bash

我总是得到结果：

0. id: 0, username: username
1. id: 1, username: username
2. id: 2, username: username
3. id: 3, username: username
4. id: 4, username: username
5. id: 5, username: username
6. id: 6, username: username
7. id: 7, username: username
8. id: 8, username: username
9. id: 9, username: username

Users
0. id: 0: ,username: 
1. id: 1: ,username: username
2. id: 2: ,username: username
3. id: 3: ,username: username
Segmentation fault

我读过O'Reilly的书“理解和使用C指针”，但我无法诊断原因。

答案

由于users是指向User的指针，*(users + i)是指向User的指针，并且是正确的指针算术。但是(*users + i)不是指针运算，它会产生你想要的结果。 当指针递增时，增量将是它指向的类型的大小。例如这里： users + 1将使用users增加sizeof(User *);这将是您系统上指针的大小。

但*users + 1将与*users增加sizeof(User);这是您定义的User结构的大小。

所以你必须在代码中将(*users + i)更改为(*(users + i))。需要额外的括号对，因为->运算符的优先级高于*运算符。

更改你的set_users和print_users函数，它将按预期工作：

void set_users()
{
    users = (User **) malloc(MAX_USERS_NUMBER * sizeof(User *));
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        *(users + i) = (User *) malloc(sizeof(User));
        (*(users + i))->id = i;
        (*(users + i))->username = (char *) malloc(strlen("username") + 1);
        strcpy((*(users + i))->username, "username");
        printf("%d. id: %d, username: %s
", i, (*(users + i))->id, (*(users + i))->username);
    }
}

void print_users()
{
    printf("
Users
");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s
", i, (*(users + i))->id, (*(users + i))->username);
    }
}

另一答案

试试这个：

#define MAX_CHARS_USERNAME 40

void set_users()
{
    static/*added*/ User* pUser;
    pUser = (User *) malloc(MAX_USERS_NUMBER * sizeof(User));
    users = &pUser;
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++)
    {
        (*users + i)->id = i;
        (*users + i)->username = (char *) malloc(MAX_CHARS_USERNAME);
        snprintf((*users + i)->username, MAX_CHARS_USERNAME, "username%d",i);
        printf("%d. id: %d, username: %s
", i, (*users + i)->id, (*users + i)->username);
    }
}

对于指向指针的指针（如果需要的话），您可能想要使用像ppUser这样的东西。并且不要忘记在使用后释放内存（以相反的顺序）！