AMOS-ss16-proj3 / amos-ss16-proj3

Repository for the wireshark plugin development project of the AMOS 16 course
GNU Affero General Public License v3.0
10 stars 12 forks source link

Protocol info column not set correctly in the Windows version #8

Open darki2002 opened 8 years ago

darki2002 commented 8 years ago

Windows does not show the description in the info column wich is set in every payload type file

darki2002 commented 8 years ago
darki2002 commented 8 years ago
darki2002 commented 8 years ago

I was able to show the Info column once, but it is unreproduceable why... (I used some printfs and then col_add_str in 0005. But wireshark crashed when searching for doip or payload types... but when scrolling arround I was able to see doip packages with correct info column set... :/)

darki2002 commented 8 years ago

I got an violation error when reading the cinfo, so I try to check weather this function is really writeable in the dissector file

darki2002 commented 8 years ago

I am on a working state now, but cannot state why it is working. This is the part of the payload-0005.c:

`void dissect_payload_0005(doip_header header, proto_item pitem, packet_info pinfo)
{ tvbuff_t
tvb; proto_tree *doip_tree;

/* set info column to description */
printf("HELLO WEORLD\n");
printf(description);
printf("\n");
printf(col_get_text(pinfo->cinfo, COL_INFO));
if(col_get_writable(pinfo->cinfo)) {
    printf("\n this is writeable! \n");
}

col_clear(pinfo->cinfo, COL_INFO);
col_add_str(pinfo->cinfo, COL_INFO, description); .....`

The wireshark console output now is (as picture, because I cannot copy from wireshark console): wireshark-console

darki2002 commented 8 years ago

After removing the debug output: `...

/* set info column to description */
/*printf("HELLO WEORLD\n");
printf(description);
printf("\n");
printf(col_get_text(pinfo->cinfo, COL_INFO));
if(col_get_writable(pinfo->cinfo)) {
    printf("\n this is writeable! \n");
}
*/
col_clear(pinfo->cinfo, COL_INFO);
col_add_str(pinfo->cinfo, COL_INFO, description);

...` I can now open wireshark again and get the write info column. I can now also search for doip or doip.payload.type == ... but whey I do a searching, the info column sets itself back to Ack-=..., Len=... etc.

darki2002 commented 8 years ago

The Error does NOT occure when using the wireshark-gtk.exe!

I will further check other wireshark versions (2.0.4 and 1.12.12).

darki2002 commented 8 years ago

Same happens with Wireshark 2.0.4 The correspongin TCP Info column is shown in the info Column.

darki2002 commented 8 years ago

Building without qt is not supported at all using cmake. (Source: Wireshark dev )

Currently we don't support building without Qt using CMake.