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

AD user object ACL owner issue #185

Open
SokratisUNI opened this issue Jun 7, 2023 · 3 comments
Open

AD user object ACL owner issue #185

SokratisUNI opened this issue Jun 7, 2023 · 3 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@SokratisUNI
Copy link

Hello,

I've noticed a weird issue which I've been trying to solve for quite some time.
On one specific account, the owner of this object was updated to another group "Domain Admins", but PingCastle reports that the old groups still have Full rights.

I have used different tools to list all permissions, inherited or not, but cannot find why this is being reported by your tool.

image

Hopefully, I am not missing something obvious here :)

@SokratisUNI
Copy link
Author

SokratisUNI commented Oct 28, 2024

So, finally had some time and created a test VM where I reproduced the above behavior.
the steps to reproduce are:

  • chose a non-admin (in this case was "kevpa") user and make him member of delegated group "Create User"
  • create a new user named "Admin" under "kevpa"
  • change owner of Admin newly created account to Domain Admins
  • make Admin member of Domain Admins

Now at this point, SDProp should do (and it does) it's magic, but PingCastle keeps showing that user kevpa is still having "indirect access".

Attached the report created with the latest PingCastle version.

Permissions assigned to group "Create usre" are:
ad_hc_contoso.net.html.zip

{400E6FE6-0762-491F-B9EA-418BDC3148FA}

Thanks for sharing this wonderfull tool!!!

@JoeDibley JoeDibley self-assigned this Oct 28, 2024
@JoeDibley JoeDibley added bug Something isn't working good first issue Good for newcomers labels Oct 28, 2024
@JoeDibley
Copy link

Hi there, thank you for the extra information and the report here! I have reproduced the issue but I cannot seem to immediately track down where exactly the bug is here but it is obvious that there is an issue with nested permissions vs adminsdholder protection (or just specific object permissions) and seems like it may be based on the container_hierarchy section not excluding objects that do not have inheritance enabled.

We will add this to the backlog and aim to fix this in the next 3.4 version. Will keep you updated.

@JoeDibley JoeDibley added this to the PingCastle 3.4 milestone Oct 29, 2024
@SokratisUNI
Copy link
Author

Thank you @JoeDibley appreciate your feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants