View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0033665 | Community | OCCT:Data Exchange | public | 2024-04-04 18:37 | 2024-07-22 09:58 |
| Reporter | lin789 | Assigned To | ichesnokova | ||
| Priority | normal | Severity | crash | ||
| Status | assigned | Resolution | open | ||
| Product Version | 7.8.0 | ||||
| Target Version | 7.9.0 | ||||
| Summary | 0033665: Data Exchange, Step Import - TransferRoots crashes for invalid STEP files | ||||
| Description | Hello everyone, I would like to report an issue that I encountered. I am new here, so if I am doing this the wrong way, please let me know :) I have played around with an example model and have attached the following: box_invalid1.step: Here, I removed the entity #/18 box_invalid2.step: Here, I removed the first parameter of entity #/11. Both files lead to a crash (segmentation fault) when calling TransferRoots, see below to reproduce. Can you please have a look at this issue? | ||||
| Steps To Reproduce | STEPControl_Reader reader; IFSelect_ReturnStatus stat = reader.ReadFile('box_invalid1.step'); reader.TransferRoots(); | ||||
| Tags | No tags attached. | ||||
| Test case number | |||||
|
|
box_invalid1.step (16,042 bytes) box_invalid2.step (16,069 bytes) |
|
|
I can't edit the issue description, so I'll add it here: When I wrote number(#) 18 as the entity number, it was automatically converted to a ticket number. The lines above should be box_invalid1.step: Here, I removed the entity number 18 box_invalid2.step: Here, I removed the first parameter of entity number 11 |
|
|
Branch CR33665 has been created by ichesnokova. SHA-1: d22aa2cac5f1f008f84dcfe7d073b6c730a16561 Detailed log of new commits: Author: ichesnok Date: Thu Apr 11 14:12:41 2024 +0100 0033665: Data Exchange, Step Import - TransferRoots crashes for invalid STEP files IsNull() check added for StepShape_FaceBound. |
|
|
Dear dpasukhi, please review branch CR33665. |
|
|
Please add test case(s) |
|
|
Branch CR33665 has been updated forcibly by ichesnokova. SHA-1: f62648121085e328b13f858fcecffecc39f4ac78 |
|
|
Dear @ichesnokova please update git commit message with details about what file are you working on. It should include that the fix for case when "..." entity is missed in some file (file corrupted) and other. Because the first mind for user is - why this changes only checking for null and nothing more... |
|
|
Branch CR33665 has been updated forcibly by ichesnokova. SHA-1: fc7b18fc358f8670ff573075dc05d458aa96d89a |
|
|
Branch CR33665 has been updated forcibly by ichesnokova. SHA-1: c73d05c0488804d05716f8d3e48ef5a9e33776cc |
|
|
Branch CR33665 has been updated forcibly by ichesnokova. SHA-1: 09061ca65a9724dc111392c0decd38aecc3855b2 |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2024-04-04 18:37 | lin789 | New Issue | |
| 2024-04-04 18:37 | lin789 | Assigned To | => ika |
| 2024-04-04 18:37 | lin789 | File Added: box_invalid1.step | |
| 2024-04-04 18:37 | lin789 | File Added: box_invalid2.step | |
| 2024-04-04 18:55 | lin789 | Note Added: 0115590 | |
| 2024-04-04 19:10 | dpasukhi | Product Version | 7.7.0 => 7.8.0 |
| 2024-04-04 20:19 | dpasukhi | Summary | TransferRoots crashes for invalid STEP files => Data Exchange, Step Import - TransferRoots crashes for invalid STEP files |
| 2024-04-04 20:19 | dpasukhi | Assigned To | ika => ichesnokova |
| 2024-04-04 20:19 | dpasukhi | Description Updated | |
| 2024-04-04 20:20 | dpasukhi | Description Updated | |
| 2024-04-04 20:20 | dpasukhi | Assigned To | ichesnokova => dpasukhi |
| 2024-04-04 20:20 | dpasukhi | Assigned To | dpasukhi => ichesnokova |
| 2024-04-04 20:20 | dpasukhi | Target Version | => 7.9.0 |
| 2024-04-11 18:00 | git | Note Added: 0115686 | |
| 2024-04-12 16:15 | ichesnokova | Assigned To | ichesnokova => dpasukhi |
| 2024-04-12 16:15 | ichesnokova | Status | new => resolved |
| 2024-04-12 16:15 | ichesnokova | Note Added: 0115696 | |
| 2024-05-31 16:07 | dpasukhi | Assigned To | dpasukhi => ichesnokova |
| 2024-05-31 16:07 | dpasukhi | Status | resolved => assigned |
| 2024-05-31 16:07 | dpasukhi | Note Added: 0115924 | |
| 2024-07-15 23:21 | git | Note Added: 0116182 | |
| 2024-07-16 10:45 | dpasukhi | Note Added: 0116186 | |
| 2024-07-19 12:31 | git | Note Added: 0116268 | |
| 2024-07-19 13:36 | git | Note Added: 0116269 | |
| 2024-07-22 09:58 | git | Note Added: 0116277 |
