Project

General

Profile

Actions
Copy link

Bug #463

closed

Assumption of length field size in workaround for wrong attribute VR

Added by Michael Onken almost 13 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Library
Target version:
3.6.2
Start date:
2012-11-22
Due date:
% Done:

100%

Estimated time:
2:00 h
Module:
dcmdata
Operating System:
Compiler:

Description

The dcmdata parser has the possibility to override the wrong VR given in a DICOM file for an attribute by the official dictionary definition. See option "-ev" in current dcmdump code.

However, the assumption about the size of the length field (2 byte versus 4 bytes) is not corrected, i.e. the parser selects the correct VR, but does keep the wrong length field size if it was different for the wrong VR for that attribute.

Instead, the parser should also consider that the length field size may have changed too, in order to parse the attribute correctly. Maybe both behaviours make sense, so think whether this is worth an extra option.

Files

Download all files
useDictForLengthField.patch (1.12 KB) useDictForLengthField.patch Uli Schlachter, 2012-11-27 15:55
useDictForLengthField2.patch (9.95 KB) useDictForLengthField2.patch Working(?) patch + tests for this feature Uli Schlachter, 2012-12-04 16:55
Actions
Copy link

Also available in: Atom PDF