diff options
author | Christopher R. Hertel <crh@samba.org> | 2002-07-01 03:42:28 +0000 |
---|---|---|
committer | Christopher R. Hertel <crh@samba.org> | 2002-07-01 03:42:28 +0000 |
commit | a2c8ca186f7261fdab49ed40949320a45868d2b5 (patch) | |
tree | e95a82f77549c17a050799fd4b89a2b01139506f /source3/CodingSuggestions | |
parent | 38788fbec5e0cf7422fb6609149d6eebacd0e093 (diff) | |
download | samba-a2c8ca186f7261fdab49ed40949320a45868d2b5.tar.gz samba-a2c8ca186f7261fdab49ed40949320a45868d2b5.tar.bz2 samba-a2c8ca186f7261fdab49ed40949320a45868d2b5.zip |
The 17-bit length field in the header contains the number of
bytes which follow the header, not the full packet size.
[Yes, the length field is either 17-bits, or (per the RFCs) it is a
16-bit length field preceeded by an 8-bit flags field of which only
the low-order bit may be used. If that bit is set, then add 65536 to
the 16-bit length field. (In other words, it's a 17-bit unsigned
length field.)
...unless, of course, the transport is native TCP [port 445] in which
case the length field *might* be 24-bits wide.]
Anyway, the change is a very minor one. We were including the four bytes
of the header in the length count and, as a result, sending four bytes of
garbage at the end of the SESSION REQUEST packet.
Small fix in function cli_session_request().
(This used to be commit d08471688b65371eb3de73b03a8ffaee86659ba0)
Diffstat (limited to 'source3/CodingSuggestions')
0 files changed, 0 insertions, 0 deletions