Skip to content

[findscu] Add response status check in C-FIND-RSP response data set#434

Merged
Enet4 merged 2 commits intomasterfrom
imp/findscu/check-status-data
Nov 9, 2023
Merged

[findscu] Add response status check in C-FIND-RSP response data set#434
Enet4 merged 2 commits intomasterfrom
imp/findscu/check-status-data

Conversation

@Enet4
Copy link
Copy Markdown
Owner

@Enet4 Enet4 commented Nov 3, 2023

Fixes a situation where the SCU would wait eternally for more messages if the SCP sent a command set with the status Pending but then responded with status 0 in the following data set.

@Enet4 Enet4 added bug This is a bug A-tool Area: tooling C-findscu Crate: dicom-findscu labels Nov 3, 2023
@Enet4 Enet4 changed the title [findscu] Add response status check in C-FIND-RSP [findscu] Add response status check in C-FIND-RSP response data set Nov 3, 2023
fixes a situation where the SCU would
wait eternally for more messages
if the SCP sent a command set with pending status
but then responded with 0 in the data set
@Enet4 Enet4 force-pushed the imp/findscu/check-status-data branch from 7e62f4d to 8dd6fd4 Compare November 5, 2023 18:29
@Enet4 Enet4 merged commit 213ccab into master Nov 9, 2023
@Enet4 Enet4 deleted the imp/findscu/check-status-data branch November 9, 2023 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tool Area: tooling bug This is a bug C-findscu Crate: dicom-findscu

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant