Kaydet (Commit) 3a4eab26 authored tarafından Caolán McNamara's avatar Caolán McNamara

ofz#3742 Integer-overflow

Change-Id: Ibb8441df04643dd52111342e6d2c2659033838bb
Reviewed-on: https://gerrit.libreoffice.org/43741Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarCaolán McNamara <caolanm@redhat.com>
Tested-by: 's avatarCaolán McNamara <caolanm@redhat.com>
üst 0667441b
...@@ -178,15 +178,13 @@ long DXFGroupReader::ReadI() ...@@ -178,15 +178,13 @@ long DXFGroupReader::ReadI()
return 0; return 0;
} }
long res = 0, nv = 1; OStringBuffer aNumber;
if (*p == '-') { if (*p == '-') {
nv=-1; aNumber.append(*p++);
p++;
} }
while ((p != end) && *p >= '0' && *p <= '9') { while ((p != end) && *p >= '0' && *p <= '9') {
res=res*10+static_cast<long>(*p-'0'); aNumber.append(*p++);
p++;
} }
while ((p != end) && (*p==0x20)) p++; while ((p != end) && (*p==0x20)) p++;
...@@ -195,7 +193,7 @@ long DXFGroupReader::ReadI() ...@@ -195,7 +193,7 @@ long DXFGroupReader::ReadI()
return 0; return 0;
} }
return res*nv; return aNumber.toString().toInt32();
} }
double DXFGroupReader::ReadF() double DXFGroupReader::ReadF()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment