Skip to content

Fix issues around dicom_ul::get_client_pdu#590

Merged
Enet4 merged 3 commits intomasterfrom
bug/ul/get_client_pdu-mitigation
Nov 2, 2024
Merged

Fix issues around dicom_ul::get_client_pdu#590
Enet4 merged 3 commits intomasterfrom
bug/ul/get_client_pdu-mitigation

Conversation

@Enet4
Copy link
Copy Markdown
Owner

@Enet4 Enet4 commented Nov 2, 2024

Summary

  • [ul] adjust get_client_pdu to prevent trailing data loss
  • [ul] optimize get_client_pdu to not allocate a Vec per reader buffer fill
  • [scpproxy] Tweak dicom-scpproxy to handle `dicom_ul closed connection errors correctly

Enet4 added 3 commits November 2, 2024 09:27
- add `read_buffer` parameter
  to hold and retain arbitrary chunks of incoming data
  in subsequent calls
- [scpproxy] adjust usage accordingly
@Enet4 Enet4 added bug This is a bug A-lib Area: library C-ul Crate: dicom-ul C-scpproxy Crate: dicom-scproxy labels Nov 2, 2024
@Enet4 Enet4 added this to the DICOM-rs 0.8 milestone Nov 2, 2024
@Enet4 Enet4 merged commit 6c44c43 into master Nov 2, 2024
@Enet4 Enet4 deleted the bug/ul/get_client_pdu-mitigation branch November 2, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-lib Area: library bug This is a bug C-scpproxy Crate: dicom-scproxy C-ul Crate: dicom-ul

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant