Anonymous | Login | 2021-01-19 02:50 MSK | ![]() |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] [ Related Changesets ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0031024 | Community | [OCCT] OCCT:Coding | public | 2019-10-03 05:42 | 2020-12-02 17:12 | ||||
Reporter | galbramc | ||||||||
Assigned To | abv | ||||||||
Priority | normal | Severity | minor | ||||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | [OCCT] 7.1.0 | ||||||||
Target Version | [OCCT] 7.5.0 | Fixed in Version | [OCCT] 7.5.0 | ||||||
Summary | 0031024: Coding - invalid left shift in BVH_RadixSorter::Perform() using -fsanitize=undefined | ||||||||
Description | Running a large of test cases with -fsanitize=undefined I see this error: opencascade-7.4/src/BVH/BVH_RadixSorter.hxx:209:34: runtime error: left shift of 16777301 by 8 places cannot be represented in type 'Standard_Integer' (aka 'int') opencascade-7.4/src/BVH/BVH_RadixSorter.hxx:209:34: runtime error: left shift of 33554432 by 8 places cannot be represented in type 'Standard_Integer' (aka 'int') One solution is to change: Standard_Interger aVoxel to Standard_Size aVoxel | ||||||||
Steps To Reproduce | Compile OCCT with -fsanitize=undefined and run the test suite. | ||||||||
Tags | No tags attached. | ||||||||
Test case number | Not needed | ||||||||
Attached Files | |||||||||
![]() |
|
(0087728) git (administrator) 2019-10-03 06:50 |
Branch CR31024 has been created by kgv. SHA-1: fa24e30851c81b9ace06f4f359e5300ad78e6ad9 Detailed log of new commits: Author: kgv Date: Thu Oct 3 06:49:59 2019 +0300 0031024: Coding - invalid left shift in BVH_RadixSorter::Perform() using -fsanitize=undefined Standard_Integer has been replaced by unsigned int for bit operations. |
(0087790) kgv (developer) 2019-10-03 19:31 |
Patch is ready for review. http://jenkins-test-12.nnov.opencascade.com:8080/view/CR31024-master-KGV [^] |
(0087802) git (administrator) 2019-10-04 10:08 |
Branch CR31024_1 has been created by abv. SHA-1: 603b5cf0930f6a0e216b388bcea446d71330f8ca Detailed log of new commits: Author: kgv Date: Thu Oct 3 06:49:59 2019 +0300 0031024: Coding - invalid left shift in BVH_RadixSorter::Perform() using -fsanitize=undefined Standard_Integer has been replaced by unsigned int for bit operations. |
(0087803) abv (manager) 2019-10-04 10:09 |
I have updated the patch to use unsigned not only in calculation of a bit mask, but also in places where it is further stored and checked. Eugene, can you please have a look at the patch in branch CR31024_1? |
(0087823) emv (developer) 2019-10-04 16:16 |
Reviewed, although tests have not been restarted after update. I have restarted the job CR31024-master-KGV with new branch. |
(0088475) git (administrator) 2019-10-23 11:51 |
Branch CR31024 has been deleted by kgv. SHA-1: fa24e30851c81b9ace06f4f359e5300ad78e6ad9 |
(0088476) git (administrator) 2019-10-23 11:51 |
Branch CR31024_1 has been deleted by kgv. SHA-1: 603b5cf0930f6a0e216b388bcea446d71330f8ca |
![]() |
|||
occt: master c2bcd983
Timestamp: 2019-10-03 03:49:59 Author: abv Committer: abv [ Details ] [ Diff ] |
0031024: Coding - invalid left shift in BVH_RadixSorter::Perform() using -fsanitize=undefined Standard_Integer has been replaced by unsigned int for bit operations. |
||
mod - src/BVH/BVH_LinearBuilder.hxx | [ Diff ] [ File ] | ||
mod - src/BVH/BVH_RadixSorter.hxx | [ Diff ] [ File ] | ||
![]() |
|||
Date Modified | Username | Field | Change |
2019-10-03 05:42 | galbramc | New Issue | |
2019-10-03 05:42 | galbramc | Assigned To | => kgv |
2019-10-03 06:25 | kgv | Relationship added | child of 0030557 |
2019-10-03 06:29 | kgv | Relationship added | child of 0027368 |
2019-10-03 06:31 | kgv | Product Version | => 7.1.0 |
2019-10-03 06:31 | kgv | Target Version | => 7.5.0 |
2019-10-03 06:31 | kgv | Summary | invalid left shift in BVH_RadixSorter.hxx using -fsanitize=undefined (OCCT 7.4) => Coding - invalid left shift in BVH_RadixSorter::Perform() using -fsanitize=undefined |
2019-10-03 06:50 | git | Note Added: 0087728 | |
2019-10-03 19:31 | kgv | Note Added: 0087790 | |
2019-10-03 19:31 | kgv | Assigned To | kgv => abv |
2019-10-03 19:31 | kgv | Status | new => resolved |
2019-10-04 09:44 | abv | Assigned To | abv => emv |
2019-10-04 10:08 | git | Note Added: 0087802 | |
2019-10-04 10:09 | abv | Note Added: 0087803 | |
2019-10-04 16:16 | emv | Note Added: 0087823 | |
2019-10-04 16:16 | emv | Assigned To | emv => bugmaster |
2019-10-04 16:16 | emv | Status | resolved => reviewed |
2019-10-23 01:07 | abv | Changeset attached | => occt master c2bcd983 |
2019-10-23 01:07 | abv | Assigned To | bugmaster => abv |
2019-10-23 01:07 | abv | Status | reviewed => verified |
2019-10-23 01:07 | abv | Resolution | open => fixed |
2019-10-23 01:18 | apn | Test case number | => Not needed |
2019-10-23 11:51 | git | Note Added: 0088475 | |
2019-10-23 11:51 | git | Note Added: 0088476 | |
2020-12-02 16:40 | emo | Fixed in Version | => 7.5.0 |
2020-12-02 17:12 | emo | Status | verified => closed |
Copyright © 2000 - 2021 MantisBT Team |