diff options
| author | Guy Harris <guy@alum.mit.edu> | 2018-01-06 19:02:16 -0800 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2018-01-07 03:03:05 +0000 |
| commit | e429e9ac7ed044c361d2306bdd1c30290daf1814 (patch) | |
| tree | b8ba5193fefd59e13e4d780a21570c4420eafc21 /epan/dissectors/packet-smb.c | |
| parent | 603bb5097d01740edc20c279d8d8f7aea1949710 (diff) | |
Do SMB_COM_SEARCH or SMB_COM_FIND results ever have a Unicode file name?
Change-Id: I11d3091bdaa7d4ecff66c238e45b265a57ef1af7
Reviewed-on: https://code.wireshark.org/review/25177
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan/dissectors/packet-smb.c')
| -rw-r--r-- | epan/dissectors/packet-smb.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/epan/dissectors/packet-smb.c b/epan/dissectors/packet-smb.c index 42fd1a64db..9732738d9d 100644 --- a/epan/dissectors/packet-smb.c +++ b/epan/dissectors/packet-smb.c @@ -5759,6 +5759,9 @@ dissect_search_dir_info(tvbuff_t *tvb, packet_info *pinfo, COUNT_BYTES_SUBR(4); /* file name */ + /* XXX - [MS-CIFS] says this is 13 *bytes*, suggesting that it's + in an OEM code page (i.e., ASCII superset), not Unicode in + UTF-16 encoding. Is it *ever* Unicode? */ fn_len = 13; fn = get_unicode_or_ascii_string(tvb, &offset, si->unicode, &fn_len, TRUE, TRUE, bcp); |