https://bugs.freedesktop.org/show_bug.cgi?id=68275

--- Comment #3 from Caolán McNamara <[email protected]> ---
Hmph, so (without debugging it properly yet) IIRC we had a valgrind warning on
an initialized nByte1, so that was presumably in TIFFReader::ConvertScanline
where there is only one use of nByte1 outside of the initial construction
so I guess that suggests that the ancient...

            if ( nByte1 == 'I' )
                pt++;

where nByte1 was a random number and never happened was accidentally the right
thing to do all along.

I wonder if it wanted to set things up to do some byteswapping foo, but I don't
see a need for any there ?

So, best it probably to revert my fix and add removing the use of the then
uninited nByte1.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to