From c9ba7a2a4d29af2542f31562cfdd64db2237aea8 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <caolanm@redhat.com>
Date: Fri, 17 Jul 2015 10:11:34 +0100
Subject: don't loop forever if pcx has short read

Change-Id: I638792417924bcb8e48995f4e789f84a2cbf4757
---
 filter/qa/cppunit/data/pcx/fail/hang-1.pcx | Bin 0 -> 897 bytes
 filter/source/graphicfilter/ipcx/ipcx.cxx  |   4 ++--
 2 files changed, 2 insertions(+), 2 deletions(-)
 create mode 100644 filter/qa/cppunit/data/pcx/fail/hang-1.pcx

diff --git a/filter/qa/cppunit/data/pcx/fail/hang-1.pcx b/filter/qa/cppunit/data/pcx/fail/hang-1.pcx
new file mode 100644
index 000000000000..73798ea56160
Binary files /dev/null and b/filter/qa/cppunit/data/pcx/fail/hang-1.pcx differ
diff --git a/filter/source/graphicfilter/ipcx/ipcx.cxx b/filter/source/graphicfilter/ipcx/ipcx.cxx
index 176299fba315..f89bc7c4b72b 100644
--- a/filter/source/graphicfilter/ipcx/ipcx.cxx
+++ b/filter/source/graphicfilter/ipcx/ipcx.cxx
@@ -224,7 +224,7 @@ void PCXReader::ImplReadBody(BitmapWriteAccess * pAcc)
     nCount = 0;
     for ( ny = 0; ny < nHeight; ny++ )
     {
-        if (m_rPCX.GetError() || m_rPCX.IsEof())
+        if (!m_rPCX.good())
         {
             nStatus = false;
             break;
@@ -248,7 +248,7 @@ void PCXReader::ImplReadBody(BitmapWriteAccess * pAcc)
                     nx--;
                     nCount--;
                 }
-                while ( nx > 0 )
+                while (nx > 0 && m_rPCX.good())
                 {
                     m_rPCX.ReadUChar( nDat );
                     if ( ( nDat & 0xc0 ) == 0xc0 )
-- 
cgit