Project

General

Profile

Actions

Patch #467

closed

Handle Cornell bug in dcmjpeg

Added by Uli Schlachter almost 13 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Category:
Library
Target version:
-
Start date:
2012-12-18
Due date:
% Done:

100%

Estimated time:
Module:
dcmjpeg
Operating System:
Compiler:

Description

See mails by Jörg Riesmeier (December 2012).

für einen Kunden habe ich heute ein DICOM-Bild mit einer offensichtlich
fehlerhaften JPEG-Lossless-Kodierung analysiert und versucht den
IJG-Code im DCMTK so zu patchen, dass das Bild korrekt verarbeitet wird.

Beschrieben wird das Problem recht zahlreich im Internet, u. a. hier:
http://java.net/jira/browse/JAI_IMAGEIO_CORE-85

Dort ist auch eine Lösung dokumentiert, an der ich mich bei meinem
Patch orientiert habe. Vielleicht schaust Du ihn Dir mal an. Wir können
auch gerne morgen mittag darüber reden, da ich wohl zum Escherweg
kommen werde :)

Schön wäre natürlich noch eine Log-Ausgabe, aber die kriege ich von
IJG-C-Code wohl nicht so ohne weiteres hin, oder?

Testen muss man das Ganze natürlich noch mit anderen (auch
fehlerfreien) Bildern. So bin ich mir nicht sicher, ob man den Fall "s == 16" nicht doch separat abhandeln sollte ... TomoVision stellt das
fehlerhafte Bild übrigens auch richtig dar. Viele andere
DICOM-Viewer/-Toolkits (einschließlich) DCMTK haben aber offensichtlich
Probleme damit.

(Newest version of the patch is attached)


Files

dcmjpeg_new.patch (1.54 KB) dcmjpeg_new.patch Uli Schlachter, 2012-12-18 14:11
dcmjpeg_buggy_cornell_encoder.patch (2.18 KB) dcmjpeg_buggy_cornell_encoder.patch Jörg Riesmeier, 2013-06-27 17:11
Actions #1

Updated by Jörg Riesmeier over 12 years ago

Here's a new version of the patch. This one has been tested with some buggy client files as well as with some correctly encoded DICOM JPEG files.

Actions #2

Updated by Marco Eichelberg about 7 years ago

  • Assignee set to Marco Eichelberg
  • % Done changed from 0 to 100

Closed by commit #b0b19e44c.

Actions #3

Updated by Marco Eichelberg about 7 years ago

  • Status changed from New to Closed
Actions

Also available in: Atom PDF