OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-05 18:16:23 +03:00
Code Packages Releases Activity

0030101: Application Framework - A mistake reading a RealArray attribute with too small value

Browse Source 
A case of reading of a very small value was skipped, for example: 2.1219957909652723e-314.
Fixed in XmlMDataStd_RealArrayDriver.cxx and XmlMDataStd_RealListDriver.cxx

// A test file bug29452.xml is extended to check this skipped case.
This commit is contained in:
vro 2019-10-14 10:29:54 +03:00 committed by bugmaster
parent d850414af1
commit 00cdefc7a2
3 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/XmlMDataStd/XmlMDataStd_RealArrayDriver.cxx
View File

@ -128,8 +128,12 @@ Standard_Boolean XmlMDataStd_RealArrayDriver::Paste
" for RealArray attribute as \"")
+ aValueStr + "\"";
myMessageDriver->Send (aMessageString, Message_Warning);
// skip the first space, if exists
while (*aValueStr != 0 && IsSpace (*aValueStr))
++aValueStr;
// skip to the next space separator
while (*aValueStr != 0 && ! IsSpace (*aValueStr)) ++aValueStr;
while (*aValueStr != 0 && ! IsSpace (*aValueStr))
++aValueStr;
}
aRealArray->SetValue(ind, aValue);
}

6
src/XmlMDataStd/XmlMDataStd_RealListDriver.cxx
View File

@ -126,8 +126,12 @@ Standard_Boolean XmlMDataStd_RealListDriver::Paste(const XmlObjMgt_Persistent&
" for RealList attribute as \"")
+ aValueStr + "\"";
myMessageDriver->Send(aMessageString, Message_Warning);
// skip the first space, if exists
while (*aValueStr != 0 && IsSpace (*aValueStr))
++aValueStr;
// skip to the next space separator
while (*aValueStr != 0 && ! IsSpace (*aValueStr)) ++aValueStr;
while (*aValueStr != 0 && !IsSpace (*aValueStr))
++aValueStr;
}
aRealList->Append(aValue);
}

4
tests/bugs/caf/bug29452
View File

@ -18,6 +18,7 @@ set res [Open $file D]
set real [GetReal D 0:1]
set rlst [GetRealList D 0:1]
set rarr [GetRealArray D 0:1]
set rarr2 [GetRealArray D 0:2]
# Check for read values
checkreal "Real value" [lindex $rlst 0] 1.1 0 1e-15
@ -25,6 +26,9 @@ checkreal "Real value" [lindex $rlst 2] 3.3 0 1e-15
checkreal "Real value" [lindex $rarr 0] 0.111 0 1e-15
checkreal "Real value" [lindex $rarr 3] 123. 0 1e-15
checkreal "Real value" [lindex $rarr 4] 3.14e12 0 1e-15
checkreal "Real value" [lindex $rarr2 0] 2 0 1e-15
checkreal "Real value" [lindex $rarr2 1] 2.1219957909652723e-314 0 1e-15
checkreal "Real value" [lindex $rarr2 2] 5 0 1e-15
foreach inf [list $real [lindex $rlst 3]] {
if { [string compare "$inf" "inf"] &&
[string compare "$inf" "infinity"] &&