From c56aef700930e117e3aca5b083ae052e3a06a362 Mon Sep 17 00:00:00 2001
From: Ian Faust <icfaust@gmail.com>
Date: Sat, 7 Sep 2024 07:51:37 +0200
Subject: [PATCH] [bug] Coverity fixes 2025.0 (#2890)

* Update kdtree_knn_classification_train_dense_default_impl.i

* Update kdtree_knn_classification_train_dense_default_impl.i

* Update kdtree_knn_classification_train_dense_default_impl.i

* Update kdtree_knn_classification_train_dense_default_impl.i
---
 .../kdtree_knn_classification_train_dense_default_impl.i  | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/cpp/daal/src/algorithms/k_nearest_neighbors/kdtree_knn_classification_train_dense_default_impl.i b/cpp/daal/src/algorithms/k_nearest_neighbors/kdtree_knn_classification_train_dense_default_impl.i
index 5fee9e640a5..7822be4cd0b 100644
--- a/cpp/daal/src/algorithms/k_nearest_neighbors/kdtree_knn_classification_train_dense_default_impl.i
+++ b/cpp/daal/src/algorithms/k_nearest_neighbors/kdtree_knn_classification_train_dense_default_impl.i
@@ -492,7 +492,7 @@ algorithmFpType KNNClassificationTrainBatchKernel<algorithmFpType, training::def
     algorithmFpType * subSamples, size_t subSampleCapacity, Status & status)
 {
     algorithmFpType samples[__KDTREE_MEDIAN_RANDOM_SAMPLE_COUNT + 1];
-    const size_t sampleCount = sizeof(samples) / sizeof(samples[0]);
+    const size_t sampleCount = __KDTREE_MEDIAN_RANDOM_SAMPLE_COUNT + 1;
 
     if (end - start <= sampleCount)
     {
@@ -589,7 +589,11 @@ algorithmFpType KNNClassificationTrainBatchKernel<algorithmFpType, training::def
 
     size_t sumMid = 0;
     size_t i      = 0;
-    for (; i < sampleCount; ++i)
+    // this iterates through the masterHist histogram and finds the median
+    // in almost all circumstances the break in the if statement will trigger
+    // unless masterHist does not contain sufficient data to exceed
+    // (end - start)/2 by the last bin.
+    for (; i < sampleCount - 1; ++i)
     {
         if (sumMid + masterHist[i] > (end - start) / 2)
         {