Mercurial > hg > orthanc
annotate OrthancFramework/Sources/DicomNetworking/Internals/GetScp.cpp @ 4224:38d446c9ee1d
fix
| author | Sebastien Jodogne <s.jodogne@gmail.com> |
|---|---|
| date | Wed, 30 Sep 2020 17:59:09 +0200 |
| parents | 7112a8af0b63 |
| children | 6f5d4bfb2c90 |
| rev | line source |
|---|---|
| 3818 | 1 /** |
| 2 * Orthanc - A Lightweight, RESTful DICOM Store | |
| 3 * Copyright (C) 2012-2016 Sebastien Jodogne, Medical Physics | |
| 4 * Department, University Hospital of Liege, Belgium | |
|
3970
4d1dcdf5c57e
fix date in some files
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3959
diff
changeset
|
5 * Copyright (C) 2017-2020 Osimis S.A., Belgium |
| 3818 | 6 * |
| 7 * This program is free software: you can redistribute it and/or | |
|
4119
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
8 * modify it under the terms of the GNU Lesser General Public License |
|
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
9 * as published by the Free Software Foundation, either version 3 of |
|
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
10 * the License, or (at your option) any later version. |
| 3818 | 11 * |
| 12 * This program is distributed in the hope that it will be useful, but | |
| 13 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 14 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
|
4119
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
15 * Lesser General Public License for more details. |
| 3818 | 16 * |
|
4119
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
17 * You should have received a copy of the GNU Lesser General Public |
|
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
18 * License along with this program. If not, see |
|
bf7b9edf6b81
re-licensing the OrthancFramework to LGPL, in order to license Stone of Orthanc under LGPL
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4044
diff
changeset
|
19 * <http://www.gnu.org/licenses/>. |
| 3818 | 20 **/ |
| 21 | |
| 22 | |
| 23 | |
| 24 | |
| 25 /*========================================================================= | |
| 26 | |
| 27 This file is based on portions of the following project: | |
| 28 | |
| 29 Program: DCMTK 3.6.0 | |
| 30 Module: http://dicom.offis.de/dcmtk.php.en | |
| 31 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
32 Copyright (C) 1994-2011, OFFIS e.V. |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
33 All rights reserved. |
| 3818 | 34 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
35 This software and supporting documentation were developed by |
| 3818 | 36 |
| 37 OFFIS e.V. | |
| 38 R&D Division Health | |
| 39 Escherweg 2 | |
| 40 26121 Oldenburg, Germany | |
| 41 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
42 Redistribution and use in source and binary forms, with or without |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
43 modification, are permitted provided that the following conditions |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
44 are met: |
| 3818 | 45 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
46 - Redistributions of source code must retain the above copyright |
| 3818 | 47 notice, this list of conditions and the following disclaimer. |
| 48 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
49 - Redistributions in binary form must reproduce the above copyright |
| 3818 | 50 notice, this list of conditions and the following disclaimer in the |
| 51 documentation and/or other materials provided with the distribution. | |
| 52 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
53 - Neither the name of OFFIS nor the names of its contributors may be |
| 3818 | 54 used to endorse or promote products derived from this software |
| 55 without specific prior written permission. | |
| 56 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
57 THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
58 "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
59 LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
60 A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
61 HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
62 SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
63 LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
64 DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
65 THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
66 (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
67 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 3818 | 68 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
69 =========================================================================*/ |
| 3818 | 70 |
| 71 | |
| 72 #include "../../PrecompiledHeaders.h" | |
| 73 #include <dcmtk/dcmnet/diutil.h> | |
| 74 #include <dcmtk/dcmdata/dcdeftag.h> | |
| 75 #include "GetScp.h" | |
| 76 | |
| 77 #include <memory> | |
| 78 | |
| 79 #include "../../DicomParsing/FromDcmtkBridge.h" | |
| 80 #include "../../DicomParsing/ToDcmtkBridge.h" | |
| 81 #include "../../Logging.h" | |
| 82 #include "../../OrthancException.h" | |
| 83 | |
| 84 #include <boost/lexical_cast.hpp> | |
| 85 | |
| 86 | |
| 87 namespace Orthanc | |
| 88 { | |
| 89 namespace | |
| 90 { | |
| 91 struct GetScpData | |
| 92 { | |
| 93 // Handle returns void. | |
| 94 IGetRequestHandler* handler_; | |
| 95 DcmDataset* lastRequest_; | |
| 96 T_ASC_Association * assoc_; | |
| 97 | |
| 98 std::string remoteIp_; | |
| 99 std::string remoteAet_; | |
| 100 std::string calledAet_; | |
|
3955
66879215cbf3
C-GET: add timeout, fix uninitalized priority, support multiple resources
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3954
diff
changeset
|
101 int timeout_; |
| 3818 | 102 |
| 4200 | 103 GetScpData() : |
| 104 handler_(NULL), | |
| 105 lastRequest_(NULL), | |
| 106 assoc_(NULL), | |
| 107 timeout_(0) | |
| 3818 | 108 { |
| 109 }; | |
| 110 }; | |
| 111 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
112 static DcmDataset *BuildFailedInstanceList(const std::string& failedUIDs) |
| 3818 | 113 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
114 if (failedUIDs.empty()) |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
115 { |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
116 return NULL; |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
117 } |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
118 else |
| 3818 | 119 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
120 std::unique_ptr<DcmDataset> rspIds(new DcmDataset()); |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
121 |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
122 if (!DU_putStringDOElement(rspIds.get(), DCM_FailedSOPInstanceUIDList, failedUIDs.c_str())) |
| 3818 | 123 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
124 throw OrthancException(ErrorCode_InternalError, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
125 "getSCP: failed to build DCM_FailedSOPInstanceUIDList"); |
| 3818 | 126 } |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
127 |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
128 return rspIds.release(); |
| 3818 | 129 } |
| 130 } | |
| 131 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
132 static void GetScpCallback( |
| 3818 | 133 /* in */ |
| 134 void *callbackData, | |
| 135 OFBool cancelled, | |
| 136 T_DIMSE_C_GetRQ *request, | |
| 137 DcmDataset *requestIdentifiers, | |
| 138 int responseCount, | |
| 139 /* out */ | |
| 140 T_DIMSE_C_GetRSP *response, | |
| 141 DcmDataset **responseIdentifiers, | |
| 142 DcmDataset **statusDetail) | |
| 143 { | |
| 144 bzero(response, sizeof(T_DIMSE_C_GetRSP)); | |
| 145 *statusDetail = NULL; | |
| 146 *responseIdentifiers = NULL; | |
| 147 | |
| 148 GetScpData& data = *reinterpret_cast<GetScpData*>(callbackData); | |
| 149 if (data.lastRequest_ == NULL) | |
| 150 { | |
| 151 DicomMap input; | |
|
4137
6774605d4341
keep only one signature for FromDcmtkBridge::ExtractDicomSummary()
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4136
diff
changeset
|
152 std::set<DicomTag> ignoreTagLength; |
|
6774605d4341
keep only one signature for FromDcmtkBridge::ExtractDicomSummary()
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
4136
diff
changeset
|
153 FromDcmtkBridge::ExtractDicomSummary(input, *requestIdentifiers, 0 /* don't truncate tags */, ignoreTagLength); |
| 3818 | 154 |
| 155 try | |
| 156 { | |
|
3959
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
157 if (!data.handler_->Handle( |
|
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
158 input, data.remoteIp_, data.remoteAet_, data.calledAet_, |
|
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
159 data.timeout_ < 0 ? 0 : static_cast<uint32_t>(data.timeout_))) |
| 3818 | 160 { |
| 161 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 162 return; | |
| 163 } | |
| 164 } | |
| 165 catch (OrthancException& e) | |
| 166 { | |
| 167 // Internal error! | |
| 168 LOG(ERROR) << "IGetRequestHandler Failed: " << e.What(); | |
| 169 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 170 return; | |
| 171 } | |
| 172 | |
| 173 data.lastRequest_ = requestIdentifiers; | |
| 174 } | |
| 175 else if (data.lastRequest_ != requestIdentifiers) | |
| 176 { | |
| 177 // Internal error! | |
| 178 LOG(ERROR) << "IGetRequestHandler Failed: Internal error lastRequestIdentifier"; | |
| 179 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 180 return; | |
| 181 } | |
| 182 | |
| 3954 | 183 if (data.handler_->GetRemainingCount() == 0) |
| 3818 | 184 { |
| 185 response->DimseStatus = STATUS_Success; | |
| 186 } | |
| 187 else | |
| 188 { | |
| 189 IGetRequestHandler::Status status; | |
| 190 | |
| 191 try | |
| 192 { | |
| 193 status = data.handler_->DoNext(data.assoc_); | |
| 194 } | |
| 195 catch (OrthancException& e) | |
| 196 { | |
| 197 // Internal error! | |
| 198 LOG(ERROR) << "IGetRequestHandler Failed: " << e.What(); | |
| 199 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 200 return; | |
| 201 } | |
| 202 | |
| 203 if (status == STATUS_Success) | |
| 204 { | |
| 3954 | 205 if (responseCount < static_cast<int>(data.handler_->GetRemainingCount())) |
| 3818 | 206 { |
| 207 response->DimseStatus = STATUS_Pending; | |
| 208 } | |
| 209 else | |
| 210 { | |
| 211 response->DimseStatus = STATUS_Success; | |
| 212 } | |
| 213 } | |
| 214 else | |
| 215 { | |
| 216 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 217 | |
| 3954 | 218 if (data.handler_->GetFailedCount() > 0 || |
| 219 data.handler_->GetWarningCount() > 0) | |
| 3818 | 220 { |
| 221 response->DimseStatus = STATUS_GET_Warning_SubOperationsCompleteOneOrMoreFailures; | |
| 222 } | |
| 3954 | 223 |
| 3818 | 224 /* |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
225 * if all the sub-operations failed then we need to generate |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
226 * a failed or refused status. cf. DICOM part 4, C.4.3.3.1 |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
227 * we choose to generate a "Refused - Out of Resources - |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
228 * Unable to perform suboperations" status. |
| 3818 | 229 */ |
| 3954 | 230 if ((data.handler_->GetFailedCount() > 0) && |
| 231 ((data.handler_->GetCompletedCount() + | |
| 232 data.handler_->GetWarningCount()) == 0)) | |
| 3818 | 233 { |
| 234 response->DimseStatus = STATUS_GET_Refused_OutOfResourcesSubOperations; | |
| 235 } | |
| 236 | |
| 3954 | 237 *responseIdentifiers = BuildFailedInstanceList(data.handler_->GetFailedUids()); |
| 3818 | 238 } |
| 239 } | |
| 240 | |
| 3954 | 241 response->NumberOfRemainingSubOperations = data.handler_->GetRemainingCount(); |
| 242 response->NumberOfCompletedSubOperations = data.handler_->GetCompletedCount(); | |
| 243 response->NumberOfFailedSubOperations = data.handler_->GetFailedCount(); | |
| 244 response->NumberOfWarningSubOperations = data.handler_->GetWarningCount(); | |
| 3818 | 245 } |
| 246 } | |
| 247 | |
| 248 OFCondition Internals::getScp(T_ASC_Association * assoc, | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
249 T_DIMSE_Message * msg, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
250 T_ASC_PresentationContextID presID, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
251 IGetRequestHandler& handler, |
| 4200 | 252 const std::string& remoteIp, |
| 253 const std::string& remoteAet, | |
| 254 const std::string& calledAet, | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
255 int timeout) |
| 3818 | 256 { |
| 257 GetScpData data; | |
| 258 data.lastRequest_ = NULL; | |
| 259 data.handler_ = &handler; | |
| 260 data.assoc_ = assoc; | |
| 261 data.remoteIp_ = remoteIp; | |
| 262 data.remoteAet_ = remoteAet; | |
| 263 data.calledAet_ = calledAet; | |
|
3955
66879215cbf3
C-GET: add timeout, fix uninitalized priority, support multiple resources
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3954
diff
changeset
|
264 data.timeout_ = timeout; |
| 3818 | 265 |
| 266 OFCondition cond = DIMSE_getProvider(assoc, presID, &msg->msg.CGetRQ, | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
267 GetScpCallback, &data, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
268 /*opt_blockMode*/ (timeout ? DIMSE_NONBLOCKING : DIMSE_BLOCKING), |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
269 /*opt_dimse_timeout*/ timeout); |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
270 |
| 3818 | 271 // if some error occured, dump corresponding information and remove the outfile if necessary |
| 272 if (cond.bad()) | |
| 273 { | |
| 274 OFString temp_str; | |
| 275 LOG(ERROR) << "Get SCP Failed: " << cond.text(); | |
| 276 } | |
| 277 | |
| 278 return cond; | |
| 279 } | |
| 280 } |