Add DICOM (Digital Imaging and Communications in Medicine) protocol support

[tmart234]

Summary

DICOM communications is used in medical imaging systems. This PR adds Scapy layers for the DICOM Upper Layer Protocol (DICOM PS3.8).

Features

• Full PDU support: A-ASSOCIATE-RQ/AC/RJ, P-DATA-TF, A-RELEASE-RQ/RP, A-ABORT
• Variable item negotiation: Application Context, Presentation Context, User Information, User Identity, Async Operations, Role Selection
• DIMSE command packets: C-ECHO, C-STORE, C-FIND, C-MOVE, C-GET

Usage

from scapy.contrib.dicom import *

# Build an A-ASSOCIATE-RQ
pkt = DICOM() / A_ASSOCIATE_RQ(
    called_ae_title=_pad_ae_title("SERVER"),
    calling_ae_title=_pad_ae_title("CLIENT"),
    variable_items=[
        DICOMVariableItem() / DICOMApplicationContext(),
        build_presentation_context_rq(1, VERIFICATION_SOP_CLASS_UID, [DEFAULT_TRANSFER_SYNTAX_UID]),
        build_user_information(max_pdu_length=16384),
    ]
)

# Build a C-MOVE-RQ
move_rq = C_MOVE_RQ(message_id=1, move_destination=b"DEST_AE")

Testing

• 59 unit tests in test/contrib/dicom.uts
• All tests pass on Python 3.9+

References

• DICOM PS3.8: https://dicom.nema.org/medical/dicom/current/output/html/part08.html

[codecov]

Codecov Report

❌ Patch coverage is 52.34708% with 335 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.62%. Comparing base (f303033) to head (f8d6ec4).

Files with missing lines	Patch %	Lines
scapy/contrib/dicom.py	52.34%	335 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4891      +/-   ##
==========================================
- Coverage   80.84%   80.62%   -0.22%     
==========================================
  Files         369      370       +1     
  Lines       90968    91671     +703     
==========================================
+ Hits        73544    73913     +369     
- Misses      17424    17758     +334     

Files with missing lines	Coverage Δ
scapy/contrib/dicom.py	52.34% <52.34%> (ø)

... and 6 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[polydroi]

Thanks for the PR. I’ve stared the unit tests. Looks good, however, could you please add type hints to your layer?

[tmart234]

Thanks for the PR. I’ve stared the unit tests. Looks good, however, could you please add type hints to your layer?

Added

[polybassa]

Besides the small Flake8 issues, LGTM.

@gpotter2 do you want to have a final look?

[tmart234]

@polybassa fixed the Flake8 issues

[gpotter2]

I trust you @polybassa !

[Copilot]

Pull request overview

This PR adds comprehensive support for the DICOM (Digital Imaging and Communications in Medicine) Upper Layer Protocol to Scapy, enabling network analysis and interaction with medical imaging systems. The implementation follows the DICOM PS3.8 specification and includes extensive packet definitions for protocol negotiation, data transfer, and DIMSE commands.

Changes:

• Added complete DICOM protocol layer implementation with PDU types, variable items, and DIMSE command packets
• Implemented a high-level DICOMSocket class for association management and DIMSE operations
• Added 59 comprehensive unit tests covering packet construction, parsing, and round-trip serialization

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.

File	Description
scapy/contrib/dicom.py	Main implementation file containing DICOM packet classes, field types, PDU definitions, DIMSE commands, and DICOMSocket for application-level operations
test/contrib/dicom.uts	Comprehensive test suite with 59 tests covering module loading, PDU construction/parsing, variable items, DIMSE packets, helper functions, and edge cases

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[polybassa]

I trust you @polybassa !

Thanks. I've added the last two copilot finding.