Mercurial > hg > orthanc
annotate Core/DicomNetworking/Internals/GetScp.cpp @ 4038:e072b4e33600
merge
| author | Sebastien Jodogne <s.jodogne@gmail.com> |
|---|---|
| date | Wed, 10 Jun 2020 12:21:22 +0200 |
| parents | 4d1dcdf5c57e |
| children |
| 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 | |
| 8 * modify it under the terms of the GNU General Public License as | |
| 9 * published by the Free Software Foundation, either version 3 of the | |
| 10 * License, or (at your option) any later version. | |
| 11 * | |
| 12 * In addition, as a special exception, the copyright holders of this | |
| 13 * program give permission to link the code of its release with the | |
| 14 * OpenSSL project's "OpenSSL" library (or with modified versions of it | |
| 15 * that use the same license as the "OpenSSL" library), and distribute | |
| 16 * the linked executables. You must obey the GNU General Public License | |
| 17 * in all respects for all of the code used other than "OpenSSL". If you | |
| 18 * modify file(s) with this exception, you may extend this exception to | |
| 19 * your version of the file(s), but you are not obligated to do so. If | |
| 20 * you do not wish to do so, delete this exception statement from your | |
| 21 * version. If you delete this exception statement from all source files | |
| 22 * in the program, then also delete it here. | |
| 23 * | |
| 24 * This program is distributed in the hope that it will be useful, but | |
| 25 * WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 26 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 27 * General Public License for more details. | |
| 28 * | |
| 29 * You should have received a copy of the GNU General Public License | |
| 30 * along with this program. If not, see <http://www.gnu.org/licenses/>. | |
| 31 **/ | |
| 32 | |
| 33 | |
| 34 | |
| 35 | |
| 36 /*========================================================================= | |
| 37 | |
| 38 This file is based on portions of the following project: | |
| 39 | |
| 40 Program: DCMTK 3.6.0 | |
| 41 Module: http://dicom.offis.de/dcmtk.php.en | |
| 42 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
43 Copyright (C) 1994-2011, OFFIS e.V. |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
44 All rights reserved. |
| 3818 | 45 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
46 This software and supporting documentation were developed by |
| 3818 | 47 |
| 48 OFFIS e.V. | |
| 49 R&D Division Health | |
| 50 Escherweg 2 | |
| 51 26121 Oldenburg, Germany | |
| 52 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
53 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
|
54 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
|
55 are met: |
| 3818 | 56 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
57 - Redistributions of source code must retain the above copyright |
| 3818 | 58 notice, this list of conditions and the following disclaimer. |
| 59 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
60 - Redistributions in binary form must reproduce the above copyright |
| 3818 | 61 notice, this list of conditions and the following disclaimer in the |
| 62 documentation and/or other materials provided with the distribution. | |
| 63 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
64 - Neither the name of OFFIS nor the names of its contributors may be |
| 3818 | 65 used to endorse or promote products derived from this software |
| 66 without specific prior written permission. | |
| 67 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
68 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
|
69 "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
|
70 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
|
71 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
|
72 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
|
73 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
|
74 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
|
75 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
|
76 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
|
77 (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
|
78 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 3818 | 79 |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
80 =========================================================================*/ |
| 3818 | 81 |
| 82 | |
| 83 #include "../../PrecompiledHeaders.h" | |
| 84 #include <dcmtk/dcmnet/diutil.h> | |
| 85 #include <dcmtk/dcmdata/dcdeftag.h> | |
| 86 #include "GetScp.h" | |
| 87 | |
| 88 #include <memory> | |
| 89 | |
| 90 #include "../../DicomParsing/FromDcmtkBridge.h" | |
| 91 #include "../../DicomParsing/ToDcmtkBridge.h" | |
| 92 #include "../../Logging.h" | |
| 93 #include "../../OrthancException.h" | |
| 94 | |
| 95 #include <boost/lexical_cast.hpp> | |
| 96 | |
| 97 | |
| 98 namespace Orthanc | |
| 99 { | |
| 100 namespace | |
| 101 { | |
| 102 struct GetScpData | |
| 103 { | |
| 104 // Handle returns void. | |
| 105 IGetRequestHandler* handler_; | |
| 106 DcmDataset* lastRequest_; | |
| 107 T_ASC_Association * assoc_; | |
| 108 | |
| 109 std::string remoteIp_; | |
| 110 std::string remoteAet_; | |
| 111 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
|
112 int timeout_; |
| 3818 | 113 |
| 114 GetScpData() | |
| 115 { | |
| 116 handler_ = NULL; | |
| 117 lastRequest_ = NULL; | |
| 118 assoc_ = NULL; | |
| 119 }; | |
| 120 }; | |
| 121 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
122 static DcmDataset *BuildFailedInstanceList(const std::string& failedUIDs) |
| 3818 | 123 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
124 if (failedUIDs.empty()) |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
125 { |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
126 return NULL; |
|
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 else |
| 3818 | 129 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
130 std::unique_ptr<DcmDataset> rspIds(new DcmDataset()); |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
131 |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
132 if (!DU_putStringDOElement(rspIds.get(), DCM_FailedSOPInstanceUIDList, failedUIDs.c_str())) |
| 3818 | 133 { |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
134 throw OrthancException(ErrorCode_InternalError, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
135 "getSCP: failed to build DCM_FailedSOPInstanceUIDList"); |
| 3818 | 136 } |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
137 |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
138 return rspIds.release(); |
| 3818 | 139 } |
| 140 } | |
| 141 | |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
142 static void GetScpCallback( |
| 3818 | 143 /* in */ |
| 144 void *callbackData, | |
| 145 OFBool cancelled, | |
| 146 T_DIMSE_C_GetRQ *request, | |
| 147 DcmDataset *requestIdentifiers, | |
| 148 int responseCount, | |
| 149 /* out */ | |
| 150 T_DIMSE_C_GetRSP *response, | |
| 151 DcmDataset **responseIdentifiers, | |
| 152 DcmDataset **statusDetail) | |
| 153 { | |
| 154 bzero(response, sizeof(T_DIMSE_C_GetRSP)); | |
| 155 *statusDetail = NULL; | |
| 156 *responseIdentifiers = NULL; | |
| 157 | |
| 158 GetScpData& data = *reinterpret_cast<GetScpData*>(callbackData); | |
| 159 if (data.lastRequest_ == NULL) | |
| 160 { | |
| 161 DicomMap input; | |
| 162 FromDcmtkBridge::ExtractDicomSummary(input, *requestIdentifiers); | |
| 163 | |
| 164 try | |
| 165 { | |
|
3959
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
166 if (!data.handler_->Handle( |
|
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
167 input, data.remoteIp_, data.remoteAet_, data.calledAet_, |
|
76a24be12912
c-get: support of transcoding
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3955
diff
changeset
|
168 data.timeout_ < 0 ? 0 : static_cast<uint32_t>(data.timeout_))) |
| 3818 | 169 { |
| 170 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 171 return; | |
| 172 } | |
| 173 } | |
| 174 catch (OrthancException& e) | |
| 175 { | |
| 176 // Internal error! | |
| 177 LOG(ERROR) << "IGetRequestHandler Failed: " << e.What(); | |
| 178 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 179 return; | |
| 180 } | |
| 181 | |
| 182 data.lastRequest_ = requestIdentifiers; | |
| 183 } | |
| 184 else if (data.lastRequest_ != requestIdentifiers) | |
| 185 { | |
| 186 // Internal error! | |
| 187 LOG(ERROR) << "IGetRequestHandler Failed: Internal error lastRequestIdentifier"; | |
| 188 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 189 return; | |
| 190 } | |
| 191 | |
| 3954 | 192 if (data.handler_->GetRemainingCount() == 0) |
| 3818 | 193 { |
| 194 response->DimseStatus = STATUS_Success; | |
| 195 } | |
| 196 else | |
| 197 { | |
| 198 IGetRequestHandler::Status status; | |
| 199 | |
| 200 try | |
| 201 { | |
| 202 status = data.handler_->DoNext(data.assoc_); | |
| 203 } | |
| 204 catch (OrthancException& e) | |
| 205 { | |
| 206 // Internal error! | |
| 207 LOG(ERROR) << "IGetRequestHandler Failed: " << e.What(); | |
| 208 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 209 return; | |
| 210 } | |
| 211 | |
| 212 if (status == STATUS_Success) | |
| 213 { | |
| 3954 | 214 if (responseCount < static_cast<int>(data.handler_->GetRemainingCount())) |
| 3818 | 215 { |
| 216 response->DimseStatus = STATUS_Pending; | |
| 217 } | |
| 218 else | |
| 219 { | |
| 220 response->DimseStatus = STATUS_Success; | |
| 221 } | |
| 222 } | |
| 223 else | |
| 224 { | |
| 225 response->DimseStatus = STATUS_GET_Failed_UnableToProcess; | |
| 226 | |
| 3954 | 227 if (data.handler_->GetFailedCount() > 0 || |
| 228 data.handler_->GetWarningCount() > 0) | |
| 3818 | 229 { |
| 230 response->DimseStatus = STATUS_GET_Warning_SubOperationsCompleteOneOrMoreFailures; | |
| 231 } | |
| 3954 | 232 |
| 3818 | 233 /* |
|
3953
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
234 * 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
|
235 * 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
|
236 * 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
|
237 * Unable to perform suboperations" status. |
| 3818 | 238 */ |
| 3954 | 239 if ((data.handler_->GetFailedCount() > 0) && |
| 240 ((data.handler_->GetCompletedCount() + | |
| 241 data.handler_->GetWarningCount()) == 0)) | |
| 3818 | 242 { |
| 243 response->DimseStatus = STATUS_GET_Refused_OutOfResourcesSubOperations; | |
| 244 } | |
| 245 | |
| 3954 | 246 *responseIdentifiers = BuildFailedInstanceList(data.handler_->GetFailedUids()); |
| 3818 | 247 } |
| 248 } | |
| 249 | |
| 3954 | 250 response->NumberOfRemainingSubOperations = data.handler_->GetRemainingCount(); |
| 251 response->NumberOfCompletedSubOperations = data.handler_->GetCompletedCount(); | |
| 252 response->NumberOfFailedSubOperations = data.handler_->GetFailedCount(); | |
| 253 response->NumberOfWarningSubOperations = data.handler_->GetWarningCount(); | |
| 3818 | 254 } |
| 255 } | |
| 256 | |
| 257 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
|
258 T_DIMSE_Message * msg, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
259 T_ASC_PresentationContextID presID, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
260 IGetRequestHandler& handler, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
261 std::string remoteIp, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
262 std::string remoteAet, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
263 std::string calledAet, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
264 int timeout) |
| 3818 | 265 { |
| 266 GetScpData data; | |
| 267 data.lastRequest_ = NULL; | |
| 268 data.handler_ = &handler; | |
| 269 data.assoc_ = assoc; | |
| 270 data.remoteIp_ = remoteIp; | |
| 271 data.remoteAet_ = remoteAet; | |
| 272 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
|
273 data.timeout_ = timeout; |
| 3818 | 274 |
| 275 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
|
276 GetScpCallback, &data, |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
277 /*opt_blockMode*/ (timeout ? DIMSE_NONBLOCKING : DIMSE_BLOCKING), |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
278 /*opt_dimse_timeout*/ timeout); |
|
620e87e9e816
c-get: fixing memory with failedUIDs_
Sebastien Jodogne <s.jodogne@gmail.com>
parents:
3818
diff
changeset
|
279 |
| 3818 | 280 // if some error occured, dump corresponding information and remove the outfile if necessary |
| 281 if (cond.bad()) | |
| 282 { | |
| 283 OFString temp_str; | |
| 284 LOG(ERROR) << "Get SCP Failed: " << cond.text(); | |
| 285 } | |
| 286 | |
| 287 return cond; | |
| 288 } | |
| 289 } |