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.

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

LsaRegisterLogonProcess SecurityMode parameter missing reserved bit #1735

Closed
riverar opened this issue Nov 13, 2023 · 2 comments
Closed

LsaRegisterLogonProcess SecurityMode parameter missing reserved bit #1735

riverar opened this issue Nov 13, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@riverar
Copy link
Collaborator

riverar commented Nov 13, 2023

Definition

NTSTATUS LsaRegisterLogonProcess(
  [in]  PLSA_STRING           LogonProcessName,
  [out] PHANDLE               LsaHandle,
  [out] PLSA_OPERATIONAL_MODE SecurityMode
);

Winmd

[DllImport("SECUR32.dll", ExactSpelling = true, PreserveSig = false)]
[SupportedOSPlatform("windows5.1.2600")]
[Documentation("https://learn.microsoft.com/windows/win32/api/ntsecapi/nf-ntsecapi-lsaregisterlogonprocess")]
public unsafe static extern NTSTATUS LsaRegisterLogonProcess([In] LSA_STRING* LogonProcessName, [Out] HANDLE* LsaHandle, [Out] uint* SecurityMode);

Docs

[out] SecurityMode
The value returned is not meaningful and should be ignored.
https://learn.microsoft.com/windows/win32/api/ntsecapi/nf-ntsecapi-lsaregisterlogonprocess

@riverar riverar added the enhancement New feature or request label Nov 13, 2023
@mikebattista mikebattista self-assigned this Nov 13, 2023
@riverar
Copy link
Collaborator Author

riverar commented Nov 13, 2023

Thanks @mikebattista!

@riverar
Copy link
Collaborator Author

riverar commented Nov 13, 2023

@mikebattista I misread and this isn't reserved. :(

The docs just simply want you to ignore what the API puts there. If this is set to NULL, you receive RPC_NT_NULL_REF_POINTER.

mikebattista added a commit that referenced this issue Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants