diff options
| author | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2012-07-28 23:51:06 +0000 |
|---|---|---|
| committer | martinm <martinm@f5534014-38df-0310-8fa8-9805f1628bb7> | 2012-07-28 23:51:06 +0000 |
| commit | ef3f1583836d5b8f4fa7b20ec2f235cf212905d7 (patch) | |
| tree | 51f9d5df70af5467d44507673d8f0383a4dc7b97 /epan/dissectors/packet-t30.c | |
| parent | 9b728f6cc4c67d071129d3b6da6f21275869f74c (diff) | |
Add a comment that VS Code Analyzer makes a very good point here -
masking the m.s.b. of the command code does mean that several of them
won't ever get detected. I couldn't readily see from the spec what the
right think to do is.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@44098 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-t30.c')
| -rw-r--r-- | epan/dissectors/packet-t30.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/epan/dissectors/packet-t30.c b/epan/dissectors/packet-t30.c index 33a756832f..b640848867 100644 --- a/epan/dissectors/packet-t30.c +++ b/epan/dissectors/packet-t30.c @@ -992,7 +992,11 @@ dissect_t30_hdlc(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) " %s - %s", val_to_str_const(octet & 0x7F, t30_facsimile_control_field_vals_short, "<unknown>"), val_to_str(octet & 0x7F, t30_facsimile_control_field_vals, "<unknown>") ); - + /* + TODO: VS Code Analysis makes a good point - several of these commands can't get detected while + the m.s.b. is masked off! (e.g. T30_FC_DTC is 0x81). I couldn't readily work out from the T.30 + spec why this masking is being done... + */ switch (octet & 0x7F) { case T30_FC_DIS: case T30_FC_DTC: |