Remove comment about file paths. And don't recompute file name length.
This commit is contained in:
parent
0db3577a84
commit
8536425d76
|
@ -76,7 +76,6 @@ X_STATUS HostPathEntry::QueryInfo(XFileInfo* out_info) {
|
||||||
X_STATUS HostPathEntry::QueryDirectory(
|
X_STATUS HostPathEntry::QueryDirectory(
|
||||||
XDirectoryInfo* out_info, size_t length, bool restart) {
|
XDirectoryInfo* out_info, size_t length, bool restart) {
|
||||||
XEASSERTNOTNULL(out_info);
|
XEASSERTNOTNULL(out_info);
|
||||||
|
|
||||||
if (length < sizeof(XDirectoryInfo)) {
|
if (length < sizeof(XDirectoryInfo)) {
|
||||||
return X_STATUS_INFO_LENGTH_MISMATCH;
|
return X_STATUS_INFO_LENGTH_MISMATCH;
|
||||||
}
|
}
|
||||||
|
@ -124,7 +123,7 @@ X_STATUS HostPathEntry::QueryDirectory(
|
||||||
do {
|
do {
|
||||||
size_t file_name_length = wcslen(ffd.cFileName);
|
size_t file_name_length = wcslen(ffd.cFileName);
|
||||||
if (((uint8_t*)&((XDirectoryInfo*)current_buf)->file_name[0]) +
|
if (((uint8_t*)&((XDirectoryInfo*)current_buf)->file_name[0]) +
|
||||||
wcslen(ffd.cFileName) > end) {
|
file_name_length > end) {
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -139,8 +138,6 @@ X_STATUS HostPathEntry::QueryDirectory(
|
||||||
current->allocation_size = 4096;
|
current->allocation_size = 4096;
|
||||||
current->attributes = (X_FILE_ATTRIBUTES)ffd.dwFileAttributes;
|
current->attributes = (X_FILE_ATTRIBUTES)ffd.dwFileAttributes;
|
||||||
|
|
||||||
// TODO: I am pretty sure you need to prefix the file paths with full path,
|
|
||||||
// not just the name.
|
|
||||||
current->file_name_length = (uint32_t)file_name_length;
|
current->file_name_length = (uint32_t)file_name_length;
|
||||||
for (size_t i = 0; i < file_name_length; ++i) {
|
for (size_t i = 0; i < file_name_length; ++i) {
|
||||||
current->file_name[i] =
|
current->file_name[i] =
|
||||||
|
|
Loading…
Reference in New Issue