1
0
mirror of https://github.com/f4exb/sdrangel.git synced 2026-08-14 23:43:43 -04:00

wdsp: Prevent potential out-of-bounds array access

Fix cppcheck warnings where array elements were accessed before validating
their indices. Reorder conditional expressions to ensure bounds checks are
performed before array accesses, taking advantage of C++ short-circuit
evaluation.

In EMNR::NP::interpM(), check the index limit before accessing xvals[idx].
In SNBA::scanFrame(), verify i is within the merit array bounds before
evaluating merit[i].

These changes improve defensive programming practices and prevent potential
undefined behavior if an index exceeds the valid range. No functional changes.

Signed-off-by: Robin Getz <rgetz503@gmail.com>
This commit is contained in:
Robin Getz
2026-07-25 19:30:38 -04:00
parent b5f216f049
commit 665f5e77ad
2 changed files with 2 additions and 2 deletions
+1 -1
View File
@@ -233,7 +233,7 @@ void EMNR::NP::interpM (
double xlhigh;
double frac;
while ((x >= xvals[idx]) && (idx < nvals - 1))
while ((idx < nvals - 1) && (x >= xvals[idx]))
idx++;
xllow = log10 (xvals[idx - 1]);
+1 -1
View File
@@ -620,7 +620,7 @@ int SNBA::scanFrame(
if (nimp > 0)
{
while (merit[i] == merit[0] && i < nimp)
while (i < nimp && merit[i] == merit[0])
i++;
}