s3: libsmb : The short name length is only a one byte field.
authorJeremy Allison <jra@samba.org>
Tue, 17 Sep 2013 02:16:52 +0000 (19:16 -0700)
committerStefan Metzmacher <metze@samba.org>
Tue, 17 Sep 2013 10:27:18 +0000 (12:27 +0200)
The next byte is "undefined" and some vendors set this to 0xff
(discovered in SNIA SDC lab tests).

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Sep 17 12:27:18 CEST 2013 on sn-devel-104

source3/libsmb/cli_smb2_fnum.c

index 18b03f3d4faae0bb666b42580f81426c4f10e41e..3253f9df9c63f08fb73bdef5ea9ae1b162d7f0d6 100644 (file)
@@ -412,7 +412,7 @@ static NTSTATUS parse_finfo_id_both_directory_info(uint8_t *dir_data,
        if (namelen > (dir_data_length - 104)) {
                return NT_STATUS_INFO_LENGTH_MISMATCH;
        }
-       slen = SVAL(dir_data + 68, 0);
+       slen = CVAL(dir_data + 68, 0);
        if (slen > 24) {
                return NT_STATUS_INFO_LENGTH_MISMATCH;
        }