Handle famous IllegalPrivatePixelSequence case

malaterre · malaterre · commit 8d32ea28006c · 2026-02-16T11:46:18.000+01:00
diff --git a/Source/DataStructureAndEncodingDefinition/gdcmExplicitDataElement.txx b/Source/DataStructureAndEncodingDefinition/gdcmExplicitDataElement.txx
@@ -203,6 +203,11 @@ std::istream &ExplicitDataElement::ReadValue(std::istream &is, bool readvalues)
       gdcm_assert( VRField & VR::OB_OW || VRField == VR::UN );
       ValueField = new SequenceOfFragments;
       }
+    else if( TagField == Tag(0x7fe1,0x1060) && VRField == VR::OB )
+      {
+      gdcmWarningMacro( "IllegalPrivatePixelSequence (7fe1,1060)" );
+      ValueField = new SequenceOfFragments;
+      }
     else
       {
       // Support cp246 conforming file:
diff --git a/Source/MediaStorageAndFileFormat/gdcmPrinter.cxx b/Source/MediaStorageAndFileFormat/gdcmPrinter.cxx
@@ -642,7 +642,7 @@ VR Printer::PrintDataElement(std::ostringstream &os, const Dicts &dicts, const D
           }
         else if ( sqf )
           {
-          gdcm_assert( t == Tag(0x7fe0,0x0010) );
+          gdcm_assert( t == Tag(0x7fe0,0x0010) || t == Tag(0x7fe1,0x1060) );
           //os << *sqf;
           }
         else if ( sqi )

Original file line number	Diff line number	Diff line change
`@@ -203,6 +203,11 @@ std::istream &ExplicitDataElement::ReadValue(std::istream &is, bool readvalues)`
`203`	`203`	`gdcm_assert( VRField & VR::OB_OW \|\| VRField == VR::UN );`
`204`	`204`	`ValueField = new SequenceOfFragments;`
`205`	`205`	`}`
	`206`	`+ else if( TagField == Tag(0x7fe1,0x1060) && VRField == VR::OB )`
	`207`	`+ {`
	`208`	`+ gdcmWarningMacro( "IllegalPrivatePixelSequence (7fe1,1060)" );`
	`209`	`+ ValueField = new SequenceOfFragments;`
	`210`	`+ }`
`206`	`211`	`else`
`207`	`212`	`{`
`208`	`213`	`// Support cp246 conforming file:`
Original file line number	Diff line number	Diff line change
`@@ -642,7 +642,7 @@ VR Printer::PrintDataElement(std::ostringstream &os, const Dicts &dicts, const D`
`642`	`642`	`}`
`643`	`643`	`else if ( sqf )`
`644`	`644`	`{`
`645`		`- gdcm_assert( t == Tag(0x7fe0,0x0010) );`
	`645`	`+ gdcm_assert( t == Tag(0x7fe0,0x0010) \|\| t == Tag(0x7fe1,0x1060) );`
`646`	`646`	`//os << *sqf;`
`647`	`647`	`}`
`648`	`648`	`else if ( sqi )`