C - указатель на указатель на структуру - ошибка сегментации - PullRequest
Новая
       23

C - указатель на указатель на структуру - ошибка сегментации

0 голосов
/ 10 сентября 2018

Я изучаю C-указатели, поэтому я попытался создать массив структур, используя обозначения указателей.Я выделяю массив указателей для структуры 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\n", i, (*users + i)->id, (*users + i)->username);
    }
}

void print_users()
{
    printf("\nUsers\n");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s\n", 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

Я прочитал книгу О'Рейли "Понимание и использование указателей C", но не могу диагностировать причину.

Ответы [ 2 ]

0 голосов
/ 11 сентября 2018

Поскольку 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\n", i, (*(users + i))->id, (*(users + i))->username);
    }
}

void print_users()
{
    printf("\nUsers\n");
    int i;
    for (i = 0; i < MAX_USERS_NUMBER; i++) 
    {
        printf("%d. id: %d: ,username: %s\n", i, (*(users + i))->id, (*(users + i))->username);
    }
}
0 голосов
/ 11 сентября 2018

Просто попробуйте это: 

#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\n", i, (*users + i)->id, (*users + i)->username);
    }
}

Для указателя на указатель (если он вообще нужен) вы можете использовать что-то вроде ppUser. И не забудьте освободить память (в обратном порядке) после использования!

...