Skip to content

[PrintNightmare] Add more exception catch in module#529

Merged
mpgn merged 1 commit into
Pennyw0rth:mainfrom
XiaoliChan:printnightmare-fix
Dec 31, 2024
Merged

[PrintNightmare] Add more exception catch in module#529
mpgn merged 1 commit into
Pennyw0rth:mainfrom
XiaoliChan:printnightmare-fix

Conversation

@XiaoliChan

@XiaoliChan XiaoliChan commented Dec 31, 2024

Copy link
Copy Markdown
Contributor

Description

Add more exception catch in printnightmare and fix kerberos authentication in module

Type of change

  • [√] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

nxc smb x.x.x.x -u valid_user -p valid_pass -M printnightmare

Screenshots (if appropriate):

  • Before:
    image
    image

  • After
    image

Checklist:

  • [√] I have ran Ruff against my changes (via poetry: poetry run python -m ruff check . --preview, use --fix to automatically fix what it can)
  • [√] My code follows the style guidelines of this project (should be covered by Ruff above)
  • [×] If reliant on third party dependencies, such as Impacket, dploot, lsassy, etc, I have linked the relevant PRs in those projects
  • [√] I have performed a self-review of my own code
  • [√] I have commented my code, particularly in hard-to-understand areas

Signed-off-by: XiaoliChan <30458572+XiaoliChan@users.noreply.github.com>
@XiaoliChan XiaoliChan changed the title [Module] Add more exception catch [PrintNightmare] Add more exception catch in Dec 31, 2024
@XiaoliChan XiaoliChan changed the title [PrintNightmare] Add more exception catch in [PrintNightmare] Add more exception catch in module Dec 31, 2024
@mpgn mpgn merged commit 131e29e into Pennyw0rth:main Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants