[poppler] Branch 'poppler-0.18' - poppler/Lexer.cc

[Leonard Rosenthol]

You need to be careful here, since there are places in the PDF spec where you MUST have an integer and returning a real would be bad.

Leonard

-----Original Message-----
From: poppler-bounces+leonardr=adobe.com at lists.freedesktop.org [mailto:poppler-bounces+leonardr=adobe.com at lists.freedesktop.org] On Behalf Of Adrian Johnson
Sent: Sunday, February 05, 2012 9:45 AM
To: poppler at lists.freedesktop.org
Subject: [poppler] Branch 'poppler-0.18' - poppler/Lexer.cc

 poppler/Lexer.cc |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit fb629ac684a7f05d95d10285807c2cf1602d28ca
Author: Adrian Johnson <ajohnson at redneon.com>
Date:   Sat Feb 4 17:13:30 2012 +1030

    Lexer: convert integer to real when overflow occurs
    
    Bug 45605
    (cherry picked from commit 38ec58ed1cb54722aece875287b847643aa2c4b8)

diff --git a/poppler/Lexer.cc b/poppler/Lexer.cc index 6250d40..98a9403 100644
--- a/poppler/Lexer.cc
+++ b/poppler/Lexer.cc
@@ -228,11 +228,13 @@ Object *Lexer::getObj(Object *obj, int objNum) {
 	break;
       }
     }
-    if (neg)
+    if (neg) {
       xi = -xi;
+      xf = -xf;
+    }
     if (unlikely(overflownInteger)) {
       if (overflownUnsignedInteger) {
-        obj->initError();
+        obj->initReal(xf);
       } else {
         obj->initUint(xui);
       }
_______________________________________________
poppler mailing list
poppler at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/poppler