MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0031073Open CASCADE[OCCT] OCCT:Codingpublic2019-10-17 05:482019-10-17 05:48
Reporterabv 
Assigned Tokgv 
PrioritynormalSeverityminor 
StatusnewResolutionopen 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.5.0*Fixed in Version 
Summary0031073: Coding - Image_PixMapData shall have data fields private
DescriptionImplementation of the class Image_PixMapData is unsafe: its fields that characterize internal structure of data (SizeBPP, SizeX, SizeY, SizeRowBytes, TopToDown) are public and can be changed arbitrarily, thus the class does not guarantee integrity of its internal state. I propose that these fields should be private, and relevant methods added to access them (read-only).

Field TopToDown should not be accessible as is (i.e. with value either 1 or UINT_MAX) but as Boolean flag. Method Image_PixMap::TopDownInc() is confusing (documented to return either +1 or -1 but returns unsigned) and does not seem useful, better shall be removed.
TagsNo tags attached.
Test case number
Attached Files

- Relationships
child of 0031048verifiedabv Visualization - runtime error reported by Clang undefined behavior sanitizer in Image_AlienPixMap::Save() 

-  Notes
There are no notes attached to this issue.

- Issue History
Date Modified Username Field Change
2019-10-17 05:48 abv New Issue
2019-10-17 05:48 abv Assigned To => kgv
2019-10-17 05:48 abv Relationship added child of 0031048


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker