mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Resolves: fdo#68275 Grayscale TIFF is imported as white

Browse Source 
this reverts 6c719c1585 "n#615223 local nbyte1
should have been class-level nByte1" which was to fix the use seen
in valgrind og the uninitialized nByte1

So additionally remove the use of the uninit nByte1 entirely

Change-Id: I5b3f4fa00d74e545f207a11a5e90935f14a23a8e
This commit is contained in:
Caolán McNamara
2013-08-22 15:06:10 +01:00
parent b904b9a487
commit 98a61a2ae1
1 changed files with 6 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
filter/source/graphicfilter/itiff/itiff.cxx
View File

@@ -63,7 +63,6 @@ private:
sal_uInt16 nDataType;
// Data taken from the TIFF tags:
sal_Bool bByteSwap; // sal_True if bits 0..7 -> 7..0 should get converted ( FILLORDER = 2 );
sal_uInt8 nByte1; // 'I', if the format is LittleEndian
sal_uLong nNewSubFile; //
sal_uLong nSubFile; //
@@ -126,12 +125,7 @@ private:
bool HasAlphaChannel() const;
public:
TIFFReader()
: pAlphaMask(0)
, pMaskAcc(0)
, nByte1(0)
{
}
TIFFReader() : pAlphaMask(0), pMaskAcc(0) {}
~TIFFReader()
{
delete pAlphaMask;
@@ -1021,8 +1015,6 @@ sal_Bool TIFFReader::ConvertScanline( sal_uLong nY )
{
sal_uLong nMinMax = ( ( 1 << 8 /*nDstBitsPerPixel*/ ) - 1 ) / ( nMaxSampleValue - nMinSampleValue );
sal_uInt8* pt = pMap[ 0 ];
if ( nByte1 == 'I' )
pt++;
for ( nx = 0; nx < nImageWidth; nx++, pt += 2 )
{
pAcc->SetPixelIndex( nY, nx, static_cast<sal_uInt8>( ((sal_uLong)*pt - nMinSampleValue) * nMinMax) );
@@ -1089,17 +1081,17 @@ void TIFFReader::MakePalCol( void )
void TIFFReader::ReadHeader()
{
sal_uInt8 nbyte2(0);
sal_uInt16 nushort(0);
sal_uInt8 nbyte1, nbyte2;
sal_uInt16 nushort;
*pTIFF >> nByte1;
if ( nByte1 == 'I' )
*pTIFF >> nbyte1;
if ( nbyte1 == 'I' )
pTIFF->SetNumberFormatInt( NUMBERFORMAT_INT_LITTLEENDIAN );
else
pTIFF->SetNumberFormatInt( NUMBERFORMAT_INT_BIGENDIAN );
*pTIFF >> nbyte2 >> nushort;
if ( nByte1 != nbyte2 || ( nByte1 != 'I' && nByte1 != 'M' ) || nushort != 0x002a )
if ( nbyte1 != nbyte2 || ( nbyte1 != 'I' && nbyte1 != 'M' ) || nushort != 0x002a )
bStatus = sal_False;
}