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

Ccache to Kirbi conversion issue, missing hostname part in sname #1264

Closed
ShutdownRepo opened this issue Feb 20, 2022 · 8 comments · Fixed by #1276
Closed

Ccache to Kirbi conversion issue, missing hostname part in sname #1264

ShutdownRepo opened this issue Feb 20, 2022 · 8 comments · Fixed by #1276

Comments

@ShutdownRepo
Copy link
Contributor

Configuration

impacket version: Impacket v0.9.25.dev1+20220208.122405.769c3196
Python version: Python 3.9.10

Issue

When obtaining a service ticket with getST, converting the .ccache ticket to .kirbi, and then doing .kirbi to .ccache results in a ticket missing a part in sname's hostname.

Screenshot from 2022-02-20 11-58-41

The issue also happens without the -impersonate flag.

I have located the error to be on the Ccache to Kirbi conversion.

Screenshot from 2022-02-19 14-20-33

Screenshot from 2022-02-19 14-41-26

Additional context

Issue raised there: #1260 (comment)

@wqreytuk
Copy link
Contributor

I don't think this is a problem, it has no negative impact on the use of tickets

@ShutdownRepo
Copy link
Contributor Author

It clearly has a negative impact, and you were the one who demonstrated it.

@wqreytuk
Copy link
Contributor

It clearly has a negative impact, and you were the one who demonstrated it.

you mean can not be imported directly?

@wqreytuk
Copy link
Contributor

wqreytuk commented Feb 20, 2022

and you have solved this problem by modifying ticketConverter.py?

@wqreytuk
Copy link
Contributor

if you mean this, then it is not ticketConvert.py's fault, my ticket after converted can be imported and used even the description of it is not correct

@ShutdownRepo
Copy link
Contributor Author

ShutdownRepo commented Feb 20, 2022

@wqreytuk worked together on the issues we faced with the service substitution and ticket conversion.

The issues are now fixed (#1202 (comment)). The initial issue with the conversion from Ccache to Kirbi can be fixed with #1265

The other issue with incomplete service substitution was fixed in commits of two separate pending PRs: 8953d05 and f71c3bb

@0xdeaddood 0xdeaddood added the in review This issue or pull request is being analyzed label Feb 22, 2022
This was referenced Mar 8, 2022
@0xdeaddood 0xdeaddood removed the in review This issue or pull request is being analyzed label Mar 11, 2022
@0xdeaddood
Copy link
Collaborator

Fixed. Thanks @ShutdownRepo and @wqreytuk!

@wqreytuk
Copy link
Contributor

Thanks

glad to make some contributions

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 a pull request may close this issue.

3 participants