I was looking through the new version of packet-megaco.c when I noticed the
following code:
case 'V':
switch(trx_type) {
case GCP_TRX_REQUEST: cmd_type = GCP_CMD_AUDITVAL_REPLY;
break;
case GCP_TRX_REPLY: cmd_type = GCP_CMD_AUDITVAL_REQ; break;
default: cmd_type = GCP_CMD_NONE; break;
}
While I don't know enough about how the new version of the dissector works
to say that is wrong, I would have expected GCP_TRX_REQUEST to set
GCM_CMD_AUDITVAL_REQ and GCP_TRX_REPLY to set GCP_CMD_AUDITVAL_REPLY.
Could someone who understands the code have a look at this please?
Thanks
Mike
_______________________________________________
Wireshark-dev mailing list
[email protected]
http://www.wireshark.org/mailman/listinfo/wireshark-dev
