Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

X64 boot fixes #115

Closed
wants to merge 21 commits into from
Closed

X64 boot fixes #115

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
1a44b66
EFER constants fix
Nov 9, 2017
be9bd17
Move RtlpUse16ByteSLists initialization into KiInitializeKernel
Nov 9, 2017
113fb47
Comment out debug code for x86 that should not run on x64
Nov 9, 2017
44b87aa
Only enable NX if it's available
Nov 9, 2017
7307870
Remove duplicate NX enabling
Nov 10, 2017
750d672
Improve readability of CPU flag checks and fix error checking wrong r…
Nov 10, 2017
b82fb5c
Quick fix: use SIZE_T instead of ULONG, because ULONG is 32-bit and o…
Nov 10, 2017
0cd5818
Deleted forgotten unused variable
Nov 10, 2017
4362171
A bunch of 'wrong size' fixes
Nov 13, 2017
42ffa6a
Fixed incorrect comparison
Nov 13, 2017
5b165b5
Fix wrong sized variable in SAC driver
Nov 13, 2017
d7a2a23
Fixed handle management errors on x64
Nov 13, 2017
18fc357
Hackplement handling missing PPEs and PDEs in page fault handler; now…
Nov 13, 2017
2a7049f
Fixed broken x86 build
Nov 13, 2017
630bc26
Fix GCC build
Nov 13, 2017
5d66904
Skip backtrace on non-x86
Nov 14, 2017
1f0c22f
Fixed wrong comparison
Nov 14, 2017
ac90cf4
Merge branch 'master' into x64-boot-fixes
tkreuzer Dec 20, 2017
28115a3
Undo obsolete change
tkreuzer Dec 20, 2017
21fb0ba
Merge branch 'master' into x64-boot-fixes
tkreuzer Dec 29, 2017
9efd916
Merge branch 'master' into x64-boot-fixes
tkreuzer Dec 29, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion ntoskrnl/fsrtl/unc.c
View file
Open in desktop
Expand Up @@ -71,7 +71,7 @@ FsRtlpIsDfsEnabled(VOID)
return TRUE;
}

return ((ULONG)KeyQueryOutput.KeyInfo.Data != 1);
return ((ULONG)KeyQueryOutput.KeyInfo.Data[0] != 1);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Data is UCHAR Data[1];, so it would need to be either return (*(PULONG)KeyQueryOutput.KeyInfo.Data != 1); or fix the structure definition above to be something like union { KEY_VALUE_PARTIAL_INFORMATION KeyInfo; struct { ULONG Header[3]; ULONG KeyValue; } } KeyQueryOutput; and return (KeyQueryOutput.KeyValue != 0); or struct KEY_VALUE_PARTIAL_INFORMATION KeyInfo; UCHAR Fill[3];} and PULONG KeyValue = (PULONG)KeyQueryOutput.KeyInfo.Data; return (*KeyValue != 1);

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aha, I see. There are some nasty little things going on here. Well, okay, will do first option. But on little-endian machine my option will work too (BTW, is PPC port still in game?).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using a portable construct would be nice in any case. Because digging each time in the code to check whether the thing will work when you switch the endianness is a pain.

}

NTSTATUS
Expand Down