Project

General

Profile

Actions

Bug #388

closed

DICOMDIR causing a crash in the destructor of DcmDicomDir

Added by Jörg Riesmeier over 14 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
High
Category:
Library
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Module:
dcmdata
Operating System:
Compiler:

Description

see http://forum.dcmtk.org/viewtopic.php?t=2942
The DICOMDIR-file seems to be coded wrongly. The following offset values occur both in OffsetOfTheNextDirectoryRecord as well as OffsetOfReferencedLowerLevelDirectoryEntity: 27540, 27972, 28404, 29130, 29424. From there the corresponding records may have been deleted twice in the destructor. Obviously this should already be detected and reported as an error during reading.

In general the robustness/error tolerance of DICOMDIR code (performance as well, see todo list) needs to be improved. Perhaps the readability too... :-)

siehe http://forum.dcmtk.org/viewtopic.php?t=2942
Die DICOMDIR-Datei scheint fehlerhaft kodiert zu sein. Die folgenden Offset-Werte kommen sowohl bei OffsetOfTheNextDirectoryRecord als auch bei OffsetOfReferencedLowerLevelDirectoryEntity vor: 27540, 27972, 28404, 29130, 29424. Daher werden vermutlich die entsprechenden Records im Destruktor doppelt gelöscht. Das sollte natürlich bereits beim Einlesen erkannt und als Fehler gemeldet werden.

Ganz allgemein sollte die Robustheit/Fehlertoleranz des DICOMDIR-Codes (neben der Performance, siehe To-do-Liste) verbessert werden. Vielleicht auch die Übersichtlichkeit ... :-)


Files

ignore_duplicates.patch (1.11 KB) ignore_duplicates.patch Uli Schlachter, 2013-03-19 12:37
Actions

Also available in: Atom PDF