Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

Adding NtOpenSection, NtClose methods and corresponding structs #189

Closed
wants to merge 11 commits into from
Closed

Adding NtOpenSection, NtClose methods and corresponding structs #189

wants to merge 11 commits into from

Conversation

hmemcpy
Copy link
Contributor

@hmemcpy hmemcpy commented Feb 15, 2016

No description provided.

/// Pointer to a <see cref="UNICODE_STRING"/> that contains the name of the object for which a handle is to be opened.
/// This must either be a fully qualified object name, or a relative path name to the directory specified by the <see cref="RootDirectory"/> member.
/// </summary>
public UNICODE_STRING ObjectName;
Copy link
Collaborator

Choose a reason for hiding this comment

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

It's it supposed to be a pointer ?

@AArnott
Copy link
Collaborator

AArnott commented Feb 15, 2016

I'm done reviewing this one. I look forward to reviewing a subsequent iteration.

@AArnott
Copy link
Collaborator

AArnott commented Feb 16, 2016

All issues are fixed in https://github.com/hmemcpy/pinvoke/pull/2. If you accept that PR, this will be updated automatically. Let me know what you think.

@hmemcpy
Copy link
Contributor Author

hmemcpy commented Feb 16, 2016

Rebased on top of master. I think it's good to go :)

AArnott added a commit that referenced this pull request Feb 16, 2016
@AArnott
Copy link
Collaborator

AArnott commented Feb 16, 2016

I'm not a fan on rebase of publicly visible/reviewed commits. So I merged the PR from before your rebase took place.

@AArnott AArnott closed this Feb 16, 2016
@hmemcpy hmemcpy deleted the ntdll branch December 13, 2017 12:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants