Skip to content

kerberos: fix S4U when using FAST - #5165

Merged
gpotter2 merged 1 commit into
secdev:masterfrom
gpotter2:krb-s4u
Sep 10, 2026
Merged

kerberos: fix S4U when using FAST#5165
gpotter2 merged 1 commit into
secdev:masterfrom
gpotter2:krb-s4u

Conversation

@gpotter2

Copy link
Copy Markdown
Member

fixes #5064

@codecov

codecov Bot commented Sep 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.81%. Comparing base (1d29984) to head (9ca6c16).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
scapy/layers/kerberos.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5165      +/-   ##
==========================================
- Coverage   80.85%   80.81%   -0.04%     
==========================================
  Files         390      390              
  Lines       97734    97736       +2     
==========================================
- Hits        79021    78985      -36     
- Misses      18713    18751      +38     
Files with missing lines Coverage Δ
scapy/layers/kerberos.py 71.79% <0.00%> (+0.10%) ⬆️

... and 30 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gpotter2
gpotter2 merged commit 190b4ba into secdev:master Sep 10, 2026
22 of 23 checks passed
@gpotter2
gpotter2 deleted the krb-s4u branch September 10, 2026 22:34
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.

KerberosClient.tgs_req: PA-S4U-X509-USER checksum keyed with session key instead of FAST reply key (KRB_AP_ERR_MODIFIED under armored S4U2self)

1 participant