Проблема SIGSEGV - PullRequest
Новая
       7

Проблема SIGSEGV

2 голосов
/ 23 мая 2010

Я разрабатываю протокол (в C) для реализации многоуровневой сетевой структуры OSI, используя cnet (http://www.csse.uwa.edu.au/cnet/). Я получаю ошибку SIGSEGV во время выполнения, однако cnet сама компилирует мои файлы исходного кода (яя не могу скомпилировать его через gcc), поэтому я не могу легко использовать любые средства отладки, такие как gdb, чтобы найти ошибку.

Вот используемые структуры и рассматриваемый код: 

typedef struct {
    char *data;
} DATA;

typedef struct {
    CnetAddr src_addr;
    CnetAddr dest_addr;
    PACKET_TYPE type;           
    DATA data;
} Packet;

typedef struct {
    int length;         
    int checksum;   
    Packet datagram;
} Frame;


static void keyboard(CnetEvent ev, CnetTimerID timer, CnetData data)
    {
    char line[80];
    int length;

    length = sizeof(line);
    CHECK(CNET_read_keyboard((void *)line, (unsigned int *)&length)); // Reads input from keyboard

    if(length > 1)
        {           /* not just a blank line */
        printf("\tsending %d bytes - \"%s\"\n", length, line);

        application_downto_transport(1, line, &length);
        }
    }

void application_downto_transport(int link, char *msg, int *length)
    {
    transport_downto_network(link, msg, length);
    }

void transport_downto_network(int link, char *msg, int *length)
    {
    Packet *p;
    DATA *d;

    p = (Packet *)malloc(sizeof(Packet));
    d = (DATA *)malloc(sizeof(DATA));

    d->data = msg;
    p->data = *d;

    network_downto_datalink(link, (void *)p, length);
    }

void network_downto_datalink(int link, Packet *p, int *length)
    {
    Frame *f;

    // Encapsulate datagram and checksum into a Frame.
    f = (Frame *)malloc(sizeof(Frame));

    f->checksum = CNET_crc32((unsigned char *)(p->data).data, *length); // Generate 32-bit CRC for the data.
    f->datagram = *p;
    f->length = sizeof(f);

    //Pass Frame to the CNET physical layer to send Frame to the require link.
    CHECK(CNET_write_physical(link, (void *)f, (size_t *)f->length));
    free(p->data);
    free(p);
    free(f);
    }

Мне удалось обнаружить, что строка: CHECK (CNET_write_physical (link, (void *) f, (size_t *) f-> length)); вызывает ошибку, но я не могу понять, почему. Любая помощь оченьоценили.

Ответы [ 2 ]

2 голосов
/ 23 мая 2010

Я думаю, это третий параметр.Попробуйте это: 

CHECK(CNET_write_physical(link, (void *)f, (size_t *)(&f->length)));

В этой строке я предполагаю, что третий параметр ожидает указатель, потому что вы приводите значение к (size_t *).Но значение, которое вы используете, является простым целочисленным значением.Таким образом, всякий раз, когда функция разыменовывает адрес, содержащийся в этом значении, это когда вы, вероятно, получаете SIGSEGV.

С кодом, который я предложил, вы приводите указатель (&f->length).Таким образом, вы должны быть в порядке, предполагая, что функция эффективно ожидает указатель на переменную, содержащую размер.

0 голосов
/ 23 мая 2010

Я вижу здесь две проблемы - sizeof(f) дает вам размер указателя , а не Frame, затем вы присваиваете size_t -типовое значение f->length, но позже приводите его к size_t*. Последнее, скорее всего, является причиной ошибки сегментации.

...