Segmentation fault

[utnubu]

Καλησπέρα,
προσπαθώ να τραβήξω κάποιες πληροφορίες από ένα sip πακέτο με τον παρακάτω κώδικα.
Αυτό που παίρνω όταν τρέχω τον κώδικα είναι segmantation fault.
Αν κάποιος μπορεί να βοηθήσει,θα είμαι ευγνώμων.

Κώδικας: Επιλογή όλων



char * tch; /* Parsing pointer for strtok function */
char * from; /* "From" pointer */
char * contact; /* "Contact" pointer */
char * user; /* "User" pointer */
char * callid; /* "Call-ID" pointer */
char * payload; /* Packet's Payload */
tch = strtok (payload,"<>;\n\"");
while (tch != NULL)
{
if (strncmp(tch, "From",4)==0) /* Comparation of first 4 chars to "From" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
from = tch;
printf ("\n SIP From: %s \n", from);
}
if (strncmp(tch, "Contact",7)==0) /* Comparation of first 7 chars to "Contact" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
contact = tch;
printf (" SIP Contact: %s \n", contact);
}
if (strncmp(tch, "Authorization",13)==0) /* Comparation of first 13 chars to "Authorization" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
user = tch;
printf (" SIP User: %s \n", user);
}
if (strncmp(tch, "Call-ID",7)==0) /* Comparation of first 7 chars to "Call-ID" and if so collect it*/
{
callid = tch;
printf (" %s \n", callid);
}
tch = strtok (NULL, "<>;\n\"");
}


[simosx]

Εδώ χρειάζεται να δώσεις ένα ολοκληρωμένο πρόγραμμα, κατά προτίμηση όσο το δυνατόν πιο μικρό. Και να μπορεί να μεταγλωττιστεί.

Ακόμα, μπορείς να δώσεις και ένα αρχείο .pcap όπου να μπορεί να χρησιμοποιήσει κάποιος μαζί με το tcpreplay για να διαπιστώσει το σφάλμα.

Εναλλακτικά, μπορείς να μεταγλωττίσεις ο ίδιος το πρόγραμμά σου με την παράμετρο -g ώστε κατά το Segmentation fault να μπορέσεις να εντοπίσεις σε ποια γραμμή συμβαίνει το σφάλμα. Δες ακόμα και το πρόγραμμα ddd για εκσφαλμάτωση (διαθέσιμο από το Κέντρο λογισμικού).

[migf1]

Καλησπέρα,
προσπαθώ να τραβήξω κάποιες πληροφορίες από ένα sip πακέτο με τον παρακάτω κώδικα.
Αυτό που παίρνω όταν τρέχω τον κώδικα είναι segmantation fault.
Αν κάποιος μπορεί να βοηθήσει,θα είμαι ευγνώμων.

Spoiler: show

Κώδικας: Επιλογή όλων



char * tch; /* Parsing pointer for strtok function */
char * from; /* "From" pointer */
char * contact; /* "Contact" pointer */
char * user; /* "User" pointer */
char * callid; /* "Call-ID" pointer */
char * payload; /* Packet's Payload */
tch = strtok (payload,"<>;\n\"");
while (tch != NULL)
{
if (strncmp(tch, "From",4)==0) /* Comparation of first 4 chars to "From" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
from = tch;
printf ("\n SIP From: %s \n", from);
}
if (strncmp(tch, "Contact",7)==0) /* Comparation of first 7 chars to "Contact" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
contact = tch;
printf (" SIP Contact: %s \n", contact);
}
if (strncmp(tch, "Authorization",13)==0) /* Comparation of first 13 chars to "Authorization" and if so collect it*/
{
tch = strtok (NULL, "<>;\n\"");
user = tch;
printf (" SIP User: %s \n", user);
}
if (strncmp(tch, "Call-ID",7)==0) /* Comparation of first 7 chars to "Call-ID" and if so collect it*/
{
callid = tch;
printf (" %s \n", callid);
}
tch = strtok (NULL, "<>;\n\"");
}


Είμαι πολύ κουρασμένος για να κοιτάξω τώρα τον κώδικα, αλλά αυτό εδώ...

Κώδικας: Επιλογή όλων


...
char * payload; /* Packet's Payload */
tch = strtok (payload,"<>;\n\"");
...


φωνάζει "seg fault"!

Το payload πρέπει να είναι non-NULL στην 1η κλήση της strtok() (και NULL στις υπόλοιπες)... εσύ το περνάς με undefined content στην 1η κλήση της strtok().

[Star_Light]

Kαι δεν χρειαζεται να το θυμάσαι και απεξω πως το 1ο ορισμα της strtok() ειναι NULL

Κώδικας: Επιλογή όλων

tch = strtok (NULL, "<>;\n\"");

Και μαλιστα το χει αλλες τοσες φορες !!!!!!!!!

Ο δεικτης οπως ειπε και ο migf1 χρησιμοποιειται χωρις να εχει αρχικοποιηθει.

Ειναι δικος σου ο κωδικας? Απο που τον πηρες??? και τι θες να κανεις με αυτον?
ΑΝ θες βοηθεια πρεπει να μοιραστεις καποιες πληροφοριες.

[utnubu]

Συγνώμη παιδιά λάθος στο copy-paste.
Το payload δεν είναι null ο κώδικας μου έχει βασιστεί στο http://www.tcpdump.org/sniffex.c
Αυτό που θέλω είναι να διατρέξω το payload ενος sip πακέτου και να τραβήξω συγκεκριμένες πληροφορίες,γι'αυτό και κάνω χρήση της strtok που το κατακερματίζει.
Όποιος ανοίξει το αρχείο που παρατείθεται θα καταλάβει αμέσως τι παίζει.
Ο κώδικας που έχω ανεβάσει είναι ένα μικρό τμήμα(χρήση της strtok) στο οποίο και μου χτυπάει το πρόβλημα.

Μορφοποιημένος Κώδικας: Επιλογή όλων

char * payload;
            int count=1;
		udp = (struct sniff_udp*)(packet + SIZE_ETHERNET + SIZE_UDP);
		size_payload = ntohs(ip->ip_len) - (size_ip + SIZE_UDP);
		
		/* define computer udp payload (segment) offset */
		payload = (u_char *)(packet + SIZE_ETHERNET + size_ip + SIZE_UDP);

	char search_reg[]="REGISTER"; /* search for register method in payload */
	char search_sip[]="Via: SIP"; /* variable to identify sip packet */
	

	if (strstr(payload, search_sip)) 
	{ 
		/* compute udp payload (segment) size */
		if (size_payload > ntohs(udp->uh_ulen))
		{
			size_payload = ntohs(udp->uh_ulen);
		}
		printf("\n SIP Packet number %d:\n", count);
		count++;
		int reg_method = 0; /* variable to identify sip method*/
		if (strstr(payload, search_reg)) 
		{ 
			reg_method = 1; /* sip method is REGISTER*/
			if ( reg_method==1)
				printf("\n REGISTER METHOD \n"); 
		}
	
		if (size_payload > 0) 
		{		  	
				char * tch;		/* Parsing pointer for strtok function */
				char * from;		/* "From" pointer */
				char * contact;		/* "Contact" pointer */
				char * user;		/* "User" pointer */
				char * callid;		/* "Call-ID" pointer */ 
				tch = strtok (payload,"<>;\n\"");
				while (tch != NULL)
				{         
					if (strncmp(tch, "From",4)==0) /* Comparation of first 4 chars to "From" and if so collect it*/
					{
						tch = strtok (NULL, "<>;\n\"");
						from = tch;                 
						printf ("\n	SIP From: %s \n", from); 
					}   
					if (strncmp(tch, "Contact",7)==0) /* Comparation of first 7 chars to "Contact" and if so collect it*/
					{
						tch = strtok (NULL, "<>;\n\"");
						contact = tch;                 
						printf ("	SIP Contact: %s \n", contact); 
					} 
					if (strncmp(tch, "Authorization",13)==0) /* Comparation of first 13 chars to "Authorization" and if so collect it*/
					{
						tch = strtok (NULL, "<>;\n\"");
						user = tch;                 
						printf ("	SIP User: %s \n", user); 
					}   
					if (strncmp(tch, "Call-ID",7)==0)  /* Comparation of first 7 chars to "Call-ID" and if so collect it*/
					{       
						callid = tch;                 
						printf (" %s \n", callid); 
					}                           
                                      tch = strtok (NULL, "<>;\n\"");
				}

[migf1]

Κάνε έναν κόπο και άλλαξε το bbcode του παραπάνω κώδικα από [code ]...[/code ] σε [z_code ]...[/z_code ] για να μορφοποιηθεί με syntax-highlighting ο κώδικας, και θα τον κοιτάξω μόλις ευκαιρήσω.

[migf1]

Α και btw, αν τον κάνεις compile με commad-line flag -g3 στον gcc, και τρέξεις μετά το εκτελέσιμο στον gdb, μόλις σκάσει το seg-fault και σταματήσει η εκτέλεση, γράψε μέσα στον gdb: bt για να κάνει backtrace και να σου δείξει σε ποια συνάρτηση και σε ποια γραμμή δημιουργήθηκε το πρόβλημα.

Αν δεν σε βοηθάει, γράψε μας εδώ σε ποια γραμμή λέει πως βγάζει το πρόβλημα.

[migf1]

Thanks για το [z_code ]! Του έριξα μια ματιά, αλλά θέλει να κάτσω να το δω συγκεντρωμένος... το έβαλες στον gdb που σου είπα, να μας πει σε ποια γραμμή σκάει;

[konnn]

Στην επιλεγμένη γραμμή λαμβάνω segmentation fault και δε μπορώ να καταλάβω γιατί.

#include <stdio.h>
//#include <stdlib.h>
//# include <time.h>

void print_board(int **);

int main(){

int board[8][8] = {
{1,0,1,0,1,0,1,0},
{0,1,0,1,0,1,0,1},
{1,0,1,0,1,0,1,0},
{0,0,0,0,0,0,0,0},
{0,0,0,0,0,0,0,0},
{0,2,0,2,0,2,0,2},
{2,0,2,0,2,0,2,0},
{0,2,0,2,0,2,0,2}
};


print_board(board);

return 0;
}
void print_board(int **b){
int i,j;
for (i=0;i<8;i++){
for(j=0;j<8;j++){
if (b[i][j]==1) printf("X");
else if (b[i][j]==2) printf("O");
else printf(".");
}
printf("\n");
}
}

[migf1]

Καλημέρα,

το πρόβλημα είναι στη δήλωση του b στην print_board()... άλλαξέ την από int **b είτε σε int b[8][8] είτε σε int b[][8] και θα είσαι OK. Από ANSI-C και μετά δεν μπορείς να δηλώνεις 2Δ πίνακες σε πρότυπα χωρίς να δηλώσεις τουλάχιστον το μήκος της κάθε γραμμής του.

Ο compiler σου βγάζει warning εκεί (αν δεν τις έχεις ήδη, ενεργοποίησέ τες με -Wall -Wextra)