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

Unexpected error with Neo4J #529

Closed
snovvcrash opened this issue Feb 1, 2022 · 7 comments
Closed

Unexpected error with Neo4J #529

snovvcrash opened this issue Feb 1, 2022 · 7 comments
Labels

Comments

@snovvcrash
Copy link
Contributor

Describe the bug
When BloodHound is set to True in cme.conf, there's an unidentified error with Neo4J while using any valid creds (and any protocol).

To Reproduce
Steps to reproduce the behavior:
cme smb <ANY_HOST> -u <ANY_USER> -p <ANY_PASSWD>

Expected behavior
Credentials should be marked as owned in BloodHound.

Screenshots
Screenshots

Crackmapexec info

  • OS: Kali 2021.4
  • Version of CME: 5.2.3
  • Installed via pipx using latest release
@mpgn mpgn added the bug label Feb 1, 2022
@mpgn
Copy link
Contributor

mpgn commented Feb 1, 2022

Hello,

Can you run with flag --verbose ?

cme --verbose smb ...

@snovvcrash
Copy link
Contributor Author

Hi @mpgn!

Thanks for the feedback.

I suppose, this is the part of --verbose that refers to Neo4j:

1

@mpgn
Copy link
Contributor

mpgn commented Feb 2, 2022

Indeed ! That because the user is not found on your neo4j database :)

Can you double check the user exist, the one you blured on screenshot.

@snovvcrash
Copy link
Contributor Author

Yep, that's right - I was trying to reproduce it with a computer account (my bad)...

But anyways, when I'm trying to set a domain admin user account as owned, there's no error message but the account is not set as owned in the DB either.

I found a reason for it: here the condition checks whether c.owned is False but in my case this owned property is null for every collected account.

So if I replace False with None it works as expected!

@mpgn
Copy link
Contributor

mpgn commented Feb 3, 2022 via email

@snovvcrash
Copy link
Contributor Author

We could also add the owned on computer accounts 😀

You mean with bh_owned.py module or by modifying the bloodhound.py helper? 🤔

I also think of editing the IF condition mentioned above to support null-valued owned property. What’s your thoughts on it?

@Cyb3rC3lt
Copy link
Contributor

Cyb3rC3lt commented Sep 15, 2022

Apologies I know this was closed but I am getting a similar issue and the user does exist in Neo4J as shown.
Also no issue using SMB as shown also. i am on version 5.2.2. Any ideas?

image

image

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants