tags 703716 + upstream patch
thanks
Hi,
I just fixed this bug in DCMTK's git repository. The commit will soon be
publicly available[0]. The patch is also attached to this mail.
The problem was that with --disable-get some code was executed which used a
wrong value for the number of supported transfer syntax and thus a NULL pointer
got interpreted as a transfer syntax.
Cheers,
Uli
[0]:
http://git.dcmtk.org/web?p=dcmtk.git;a=commitdiff;h=29d8bca29c6c62c9f57618a58faf3b31cd12609b
diff --git a/dcmqrdb/libsrc/dcmqrsrv.cc b/dcmqrdb/libsrc/dcmqrsrv.cc
index 2b4239f..d0e77fe 100644
--- a/dcmqrdb/libsrc/dcmqrsrv.cc
+++ b/dcmqrdb/libsrc/dcmqrsrv.cc
@@ -796,7 +796,7 @@ OFCondition DcmQueryRetrieveSCP::negotiateAssociation(T_ASC_Association * assoc)
cond = ASC_acceptContextsWithPreferredTransferSyntaxes(
assoc->params,
dcmAllStorageSOPClassUIDs, numberOfAllDcmStorageSOPClassUIDs,
- (const char**)transferSyntaxes, DIM_OF(transferSyntaxes));
+ (const char**)transferSyntaxes, numTransferSyntaxes);
if (cond.bad()) {
DCMQRDB_ERROR("Cannot accept presentation contexts: " << DimseCondition::dump(temp_str, cond));
}
