UpsilonNumworks/Upsilon
1
0
Fork 0
mirror of https://github.com/UpsilonNumworks/Upsilon.git synced 2026-01-19 00:37:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

[apps/shared] CurveView: add an assertion on the number of labels to

Browse Source 
avoid reading out of bound memory
This commit is contained in:
Émilie Feral
2019-10-18 11:06:23 +02:00
committed by LeaNumworks
parent 2ff0b789f9
commit f6cfcab4e5
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
apps/shared/curve_view.cpp
View File

@@ -167,7 +167,9 @@ int CurveView::numberOfLabels(Axis axis) const {
float labelStep = 2.0f * gridUnit(axis);
float minLabel = std::ceil(min(axis)/labelStep);
float maxLabel = std::floor(max(axis)/labelStep);
return maxLabel - minLabel + 1;
int numberOfLabels = maxLabel - minLabel + 1;
assert(numberOfLabels <= (axis == Axis::Horizontal ? k_maxNumberOfXLabels : k_maxNumberOfYLabels));
return numberOfLabels;
}
void CurveView::computeLabels(Axis axis) {