Frage
Ich bin ein Protokoll der Gestaltung (in C) zu implementieren, die OSI-Netzwerk-Struktur geschichtet, mit CNET ( http://www.csse.uwa.edu.au/cnet/ ). Ich bin einen SIGSEGV Fehler zur Laufzeit bekommen, aber CNET meiner Quellcodedateien kompiliert selbst (ich es durch gcc nicht kompilieren können), damit ich nicht so leicht keine Debugging-Tools wie gdb verwenden, um den Fehler zu finden.
Hier die Strukturen verwendet wird, und der Code in Frage:
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);
}
Ich konnte feststellen, dass die Zeile: CHECK (CNET_write_physical (Link, (void *) f, (size_t *) f-> Länge)); die segfault verursacht, aber ich kann nicht herausfinden, warum. Jede Hilfe wird sehr geschätzt.
Lösung
Ich denke, ist der dritte Parameter. Versuchen Sie folgendes:
CHECK(CNET_write_physical(link, (void *)f, (size_t *)(&f->length)));
In dieser Zeile gehe ich davon aus dritten Parameter erwartet einen Zeiger, weil Sie einen Wert in einem (size_t *)
werfen. Aber der Wert, den Sie Gießen ist eine einfache Integer-Wert. Also, wenn die Funktion Dereferenzierungen die Adresse in diesem Wert enthalten ist, ist, wenn Sie wahrscheinlich eine SIGSEGV erhalten.
Mit dem Code, den ich vorgeschlagen, Sie werfen einen Zeiger (&f->length)
. So sollten Sie gut zu gehen, vorausgesetzt, die Funktion effektiv einen Zeiger auf eine Variable erwartet wird eine Größe zu halten.
Andere Tipps
I see two problems here - sizeof(f)
gives you size of a pointer not the Frame
, then you assign size_t
-typed value to f->length
, but later cast it to size_t*
. The latter is most probably the cause of the segmentation fault.