View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0022947 | Community | OCCT:Data Exchange | public | 2012-01-30 20:10 | 2012-04-16 12:36 |
| Reporter | Pawel | Assigned To | bugmaster | ||
| Priority | normal | Severity | minor | ||
| Status | closed | Resolution | fixed | ||
| Platform | A | OS | L | ||
| Product Version | 6.5.2 | ||||
| Target Version | 6.5.3 | Fixed in Version | 6.5.3 | ||
| Summary | 0022947: XCAFPrs_AISObject.cxx: array index out of bounds ... | ||||
| Description | In the line 96 an array index exceeding array size is used: B.Get(X[1], Y[1], Z[1], X[2], Y[2], Z[2]); Change the line to: B.Get(X[0], Y[0], Z[0], X[1], Y[1], Z[1]); Besides, only 10 fields of the 'Indx' array are assigned in lines 93-96 Indx [0]=1;Indx [1]=2;Indx [2]=3;Indx [3]=4;Indx [4]=5;Indx [5]=6;Indx [6]=7; Indx [7]=8;Indx [8]=1;Indx [9]=2;Indx [10]=6;Indx [10]=5;Indx [10]=3; Indx [10]=4;Indx [10]=8;Indx [10]=7; | ||||
| Tags | No tags attached. | ||||
| Test case number | test case is not required | ||||
|
|
The dirty code has been copied from AIS_Shape.cxx while #define that activated good part of that code has been lost... To be fixed. Please consider possibility of re-using implementation of method DisplayBox() of AIS_Shape instead of having a copy. |
|
|
DisplayBox function has been removed from XCAFPrs_AISObject.cxx Branch http://svn/svn/occt/branches/OCC22947 is ready to be reviewed. Dear Andrey, Please review. |
|
|
No remarks, please test |
|
|
Dear BugMaster, Workbench KAS:dev:mkv-22947-occt was created from SVN branch http://svn/svn/occt/branches/OCC22947 (and mkv-22947-products from trunk) and compiled on Linux platform. |
|
|
Dear BugMaster, Workbench mkv-22947-occt was created from SVN branch http://svn/svn/occt/branches/OCC22947 (and mkv-22947-products from trunk) and compiled on Linux platform. There are not regressions in mkv-22947-products regarding to KAS:dev:products-20120127-opt See results in /QADisk/occttests/results/KAS/dev/mkv-22947-products_14022012/lin See reference results in /QADisk/occttests/results/KAS/dev/products-20120127-opt_28012012/lin See test cases in /QADisk/occttests/tests/ED N.B. In order to launch testing case you can make use the following instructions http://doc/doku.php?id=occt:certification |
|
|
Integrated into trunk of repository Date: 2012-02-17 11:19:03 +0400 (Fri, 17 Feb 2012) New Revision: 10492 Modified: trunk/src/XCAFPrs/XCAFPrs_AISObject.cxx |
|
|
Verified. Please close. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-01-30 20:10 | Pawel | New Issue | |
| 2012-01-30 20:10 | Pawel | Assigned To | => gka |
| 2012-01-31 09:23 | abv | Relationship added | related to 0022948 |
| 2012-01-31 10:21 | abv | Note Added: 0019319 | |
| 2012-01-31 10:21 | abv | Assigned To | gka => dbv |
| 2012-01-31 10:21 | abv | Status | new => assigned |
| 2012-02-14 11:28 | dbv | Note Added: 0019541 | |
| 2012-02-14 11:28 | dbv | Assigned To | dbv => abv |
| 2012-02-14 11:28 | dbv | Status | assigned => resolved |
| 2012-02-14 11:39 | abv | Note Added: 0019542 | |
| 2012-02-14 11:39 | abv | Assigned To | abv => dbv |
| 2012-02-14 11:39 | abv | Status | resolved => reviewed |
| 2012-02-14 13:32 |
|
Assigned To | dbv => mkv |
| 2012-02-16 13:27 |
|
Test case number | => test case is not required |
| 2012-02-16 15:23 |
|
Note Added: 0019600 | |
| 2012-02-16 18:10 |
|
Note Added: 0019604 | |
| 2012-02-16 18:11 |
|
Note Edited: 0019604 | |
| 2012-02-16 18:11 |
|
Status | reviewed => tested |
| 2012-02-17 11:20 | bugmaster | Note Added: 0019608 | |
| 2012-02-17 11:20 | bugmaster | Status | tested => verified |
| 2012-02-17 11:20 | bugmaster | Resolution | open => fixed |
| 2012-02-17 11:41 | bugmaster | Assigned To | mkv => Pawel |
| 2012-02-17 11:41 | bugmaster | Target Version | => 6.5.3 |
| 2012-03-29 17:26 | bugmaster | Changeset attached | => occt master caa1174c |
| 2012-04-16 12:35 | Pawel | Assigned To | Pawel => bugmaster |
| 2012-04-16 12:36 | Pawel | Note Added: 0020393 |
