Actions
Bug #816
closed
DcmVR::setVR() should check whether internal VR is acceptable
Start date:
2018-03-09
Due date:
% Done:
100%
Estimated time:
Module:
dcmdata
Operating System:
Compiler:
Description
Commit 40c9085 fixes a heap overflow issue that is caused by an invalid dataset. This is done by making sure that internal VRs (such as "PixelData" and "OverlayData") are handled appropriately in DcmItem::newDicomElement().
However, this is just a workaround. It would probably make more sense to check for these internal VRs already in DcmVR::setVR(onst char*), which is also called by the corresponding constructor of this class. Special attention should be paid that required mappings from internal VR names to the associated enum (e.g. in dump2dcm?) still works.
Updated by Jörg Riesmeier over 7 years ago
- Status changed from New to Closed
- Assignee set to Jörg Riesmeier
- % Done changed from 0 to 100
Fixed by commit 0c4cf6e.
Updated by
Actions