View Issue Details

IDProjectCategoryView StatusLast Update
0031217CommunityOCCT:Modeling Algorithmspublic2023-08-08 14:47
Reporterdenix56 Assigned Toakaftasev  
PrioritynormalSeveritycrash 
Status verifiedResolutionfixed 
Product Version7.4.0 
Target Version7.7.3 
Summary0031217: Modeling Algorithms - Exception is raised in GeomFill_SectionPlacement when parallel path and section
DescriptionThe following code from GeomFill_SectionPlacement raises exception due to inconsistency with 7.4.0 changes. In 7.4.0 Extrema_ExtCC has IsParallel method that we need to check, otherwise exception when accessing points will be raised.
 
 Extrema_ExtCC Ext(Path->Curve(), myAdpSection,
                Path->FirstParameter(), Path->LastParameter(),
                myAdpSection.FirstParameter(),
                myAdpSection.LastParameter(),
                Path->Resolution(Tol/100),
                myAdpSection.Resolution(Tol/100));
      if (Ext.IsDone()) {
        Extrema_POnCurv P1, P2;
        for (ii=1; ii<=Ext.NbExt(); ii++) {
          distaux = sqrt (Ext.SquareDistance(ii));
          Ext.Points(ii, P1, P2); // HERE THE EXCEPTION
Steps To ReproduceNot possible
TagsNo tags attached.
Test case number

Relationships

related to 0031249 newmsv Modeling Algorithms - Invalid memory access with parallel lines and GeomAPI_ExtremaCurveCurve 

Activities

akaftasev

2020-02-05 15:42

developer   ~0090302

Please, could you reproduce a use case in which an exception arise and attach it here

git

2020-02-07 13:34

administrator   ~0090392

Branch CR31217 has been created by akaftasev.

SHA-1: 17561f1b3eac0ff7b48fe1403531259eb39489f9


Detailed log of new commits:

Author: KafAndrey
Date: Fri Feb 7 13:34:03 2020 +0300

    edit code

Author: KafAndrey
Date: Wed Feb 5 13:45:40 2020 +0300

    0031217: start testing bug

denix56

2020-03-25 12:26

reporter   ~0091236

Last edited: 2020-03-25 12:27

Sorry, but I do not work on this project anymore and I have no possibility to reproduce it.
Could you please change the assignment from me, so I will stop receiving e-mails?

git

2020-04-13 12:36

administrator   ~0091532

Branch CR31217 has been updated by akaftasev.

SHA-1: eca7b5ea4afc8414e59e1b316468215b1c30f323


Detailed log of new commits:

Author: andrey.kaftasev@opencascade.com
Date: Mon Apr 13 12:36:47 2020 +0300

    0031217: Modeling Algorithms - Exception is raised in GeomFill_SectionPlacement when parallel path and SectionAxis
    
    Now code protected from crashes when IsParallel

akaftasev

2020-04-13 12:37

developer   ~0091533

We can not reproduce this bug, but now code protected from crashes when IsParallel

msv

2020-04-15 21:41

developer   ~0091572

Please rebase the fix on current master.
I think the changes in QABugs are redundant. Remove them.
Run tests on Jenkins.
After tests are OK change to resolved again.

git

2020-04-16 12:44

administrator   ~0091587

Branch CR31217 has been updated by akaftasev.

SHA-1: 76759f4784cb1bff695af9ef3119f7b8bb7b752f


Detailed log of new commits:

Author: andrey.kaftasev@opencascade.com
Date: Thu Apr 16 12:21:22 2020 +0300

    Remove redundant changes on QABugs

git

2020-04-16 14:50

administrator   ~0091594

Branch CR31217 has been updated by akaftasev.

SHA-1: 278d163d3fa8a791d7ec4d3b76554b13199f68af


Detailed log of new commits:

Author: akaftasev
Date: Thu Apr 16 14:43:42 2020 +0300

    add deleted incude-file

git

2023-08-01 16:00

administrator   ~0113820

Branch CR31217 has been updated forcibly by andrey.kaftasev_159541.

SHA-1: b42da8f8eec3affc1a9cc494313cbc177db17650

akaftasev

2023-08-02 11:18

developer   ~0113844

Please, review branch:
OCCT - CR31217

Results of testing:
http://jenkins-test-10.nnov.opencascade.com/view/CR31217-master-akaftasev/view/COMPARE/

dpasukhi

2023-08-02 12:07

administrator   ~0113847

Dear bugmaster,
please integrate:
OCCT: CR31217
PROD: NO

jokwajeb

2023-08-08 13:59

administrator   ~0113930

Combination -
OCCT branch : SHA - IR-2023-08-04
Products branch : SHA - master
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Ubuntu2004-64:
OCCT
Total CPU difference: 18344.600000000195 / 18390.260000000217 [-0.25%]
Products
Total CPU difference: 12102.810000000003 / 12161.439999999999 [-0.48%]
Windows-64-VC142:
OCCT
Total CPU difference: 20332.1875 / 20298.859375 [+0.16%]
Products
Total CPU difference: 14548.515625 / 14604.578125 [-0.38%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

git

2023-08-08 14:47

administrator   ~0113938

Branch CR31217 has been deleted by mnt.

SHA-1: b42da8f8eec3affc1a9cc494313cbc177db17650

Related Changesets

occt: master b3242585

2023-08-01 16:00:20

akaftasev


Committer: jokwajeb Details Diff
0031217: Modeling Algorithms - Exception is raised in GeomFill_SectionPlacement when parallel path and SectionAxis

Added protection from crashes when IsParallel
Affected Issues
0031217
mod - src/GeomFill/GeomFill_SectionPlacement.cxx Diff File

Issue History

Date Modified Username Field Change
2019-12-04 16:01 denix56 New Issue
2019-12-04 16:01 denix56 Assigned To => msv
2019-12-17 13:35 kgv Product Version => 7.4.0
2019-12-17 13:35 kgv Summary Exception is raised in GeomFill_SectionPlacement when parallel path and section => Modeling Algorithms - Exception is raised in GeomFill_SectionPlacement when parallel path and section
2019-12-17 13:44 kgv Relationship added related to 0031249
2020-01-29 16:12 msv Assigned To msv => akaftasev
2020-01-29 16:12 msv Status new => assigned
2020-02-05 15:42 akaftasev Note Added: 0090302
2020-02-05 15:46 msv Assigned To akaftasev => denix56
2020-02-05 15:46 msv Status assigned => feedback
2020-02-07 13:34 git Note Added: 0090392
2020-03-25 12:26 denix56 Note Added: 0091236
2020-03-25 12:27 denix56 Note Edited: 0091236
2020-04-13 12:36 git Note Added: 0091532
2020-04-13 12:37 akaftasev Note Added: 0091533
2020-04-13 17:12 akaftasev Assigned To denix56 => msv
2020-04-13 17:12 akaftasev Status feedback => resolved
2020-04-13 17:12 akaftasev Steps to Reproduce Updated
2020-04-15 21:41 msv Note Added: 0091572
2020-04-15 21:41 msv Assigned To msv => akaftasev
2020-04-15 21:41 msv Status resolved => assigned
2020-04-16 12:44 git Note Added: 0091587
2020-04-16 14:50 git Note Added: 0091594
2023-08-01 16:00 git Note Added: 0113820
2023-08-01 17:10 dpasukhi Target Version => 7.7.3
2023-08-02 11:18 akaftasev Assigned To akaftasev => dpasukhi
2023-08-02 11:18 akaftasev Status assigned => resolved
2023-08-02 11:18 akaftasev Note Added: 0113844
2023-08-02 12:07 dpasukhi Assigned To dpasukhi => bugmaster
2023-08-02 12:07 dpasukhi Status resolved => reviewed
2023-08-02 12:07 dpasukhi Note Added: 0113847
2023-08-08 13:59 jokwajeb Note Added: 0113930
2023-08-08 14:23 jokwajeb Changeset attached => occt master b3242585
2023-08-08 14:23 akaftasev Assigned To bugmaster => akaftasev
2023-08-08 14:23 akaftasev Status reviewed => verified
2023-08-08 14:23 akaftasev Resolution open => fixed
2023-08-08 14:47 git Note Added: 0113938