View Issue Details

IDProjectCategoryView StatusLast Update
0031073Open CASCADEOCCT:Codingpublic2023-08-01 15:09
ReporterabvAssigned Tovpozdyayev 
Status newResolutionopen 
Target VersionUnscheduled 
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


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


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
2020-08-28 14:28 kgv Target Version 7.5.0 => 7.6.0
2021-08-24 14:59 kgv Target Version 7.6.0 => 7.7.0
2022-08-17 12:01 kgv Target Version 7.7.0 => 7.8.0
2022-10-19 15:56 smoskvin Assigned To kgv => vpozdyayev
2023-08-01 15:09 dpasukhi Target Version 7.8.0 => Unscheduled