Skip to content

Add registry abstraction layer to ease the manipulation of [MS-RRP] RPC#4936

Open
Ebrix wants to merge 29 commits intosecdev:masterfrom
Ebrix:master
Open

Add registry abstraction layer to ease the manipulation of [MS-RRP] RPC#4936
Ebrix wants to merge 29 commits intosecdev:masterfrom
Ebrix:master

Conversation

@Ebrix
Copy link
Contributor

@Ebrix Ebrix commented Feb 26, 2026

Checklist:

  • If you are new to Scapy: I have checked CONTRIBUTING.md (esp. section submitting-pull-requests)
  • I squashed commits belonging together
  • I added unit tests or explained why they are not relevant
  • The unit tests are part of the Scapy-Red project, they use the RPC.
  • I executed the regression tests (using tox)
  • If the PR is still not finished, please create a Draft Pull Request

The PR does 2 things :

    1. create a "windows" folder to move windows related code (typically move Windows security objects from smb2 to windows/security.py)
    1. add a registry.py file which gives "high" level abstraction of the ms_rrp.py RPC to allow the dev to easily interact with the RPC

Some import paths are modified

@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

❌ Patch coverage is 18.95425% with 496 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.23%. Comparing base (c3c62ec) to head (f9feafa).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
scapy/layers/windows/registry.py 0.00% 310 Missing ⚠️
scapy/layers/msrpce/raw/ms_rrp.py 0.00% 151 Missing ⚠️
scapy/layers/windows/security.py 76.38% 34 Missing ⚠️
scapy/modules/ldaphero.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4936      +/-   ##
==========================================
- Coverage   80.69%   80.23%   -0.46%     
==========================================
  Files         370      374       +4     
  Lines       91759    92447     +688     
==========================================
+ Hits        74045    74176     +131     
- Misses      17714    18271     +557     
Files with missing lines Coverage Δ
scapy/layers/msrpce/mspac.py 90.78% <100.00%> (ø)
scapy/layers/smb2.py 90.28% <ø> (+1.86%) ⬆️
scapy/layers/smbclient.py 74.41% <100.00%> (+0.03%) ⬆️
scapy/layers/smbserver.py 72.82% <100.00%> (+0.03%) ⬆️
scapy/layers/windows/__init__.py 100.00% <100.00%> (ø)
scapy/modules/ticketer.py 46.01% <100.00%> (+4.37%) ⬆️
scapy/modules/ldaphero.py 0.00% <0.00%> (ø)
scapy/layers/windows/security.py 76.38% <76.38%> (ø)
scapy/layers/msrpce/raw/ms_rrp.py 0.00% <0.00%> (ø)
scapy/layers/windows/registry.py 0.00% <0.00%> (ø)

... and 12 files with indirect coverage changes

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

@Ebrix Ebrix marked this pull request as ready for review February 26, 2026 13:05
@gpotter2 gpotter2 self-assigned this Feb 26, 2026
Copy link
Member

@gpotter2 gpotter2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for the PR and your interest in Scapy ! This is very cool work !!!!

I have minor comments, very picky, but overall it looks great 👍

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