From 28f4e22bd75a50ad29be0a34d09b760dec4623c4 Mon Sep 17 00:00:00 2001 From: "Shelia J." <12310645@mail.sustech.edu.cn> Date: Thu, 21 May 2026 19:26:40 +0800 Subject: [PATCH] Merge pull request #29084 from SheliaJimenez:dnn-bug-fix Fixed gatherND offset calculation #29084 Bug fix: fixed segmentation fault when running ALIKED onnx model with ENGINE_NEW. ### Description issue: when running ALIKED onnx model with ENGINE_NEW, segmentation fault happens during the inference. fix: fixed the offset calculation in gatherND.cpp, so gather does not access out-of-bound memory. ### Pull Request Readiness Checklist See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request - [x] I agree to contribute to the project under Apache 2 License. - [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV - [x] The PR is proposed to the proper branch - [ ] There is a reference to the original bug report and related work - [ ] There is accuracy test, performance test and test data in opencv_extra repository, if applicable Patch to opencv_extra has the same branch name. - [ ] The feature is well documented and sample code can be built with the project CMake --- modules/dnn/src/layers/gatherND.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/modules/dnn/src/layers/gatherND.cpp b/modules/dnn/src/layers/gatherND.cpp index 865a846230..5ff5303732 100644 --- a/modules/dnn/src/layers/gatherND.cpp +++ b/modules/dnn/src/layers/gatherND.cpp @@ -151,6 +151,10 @@ public: const int inner_size = out.total() / outer_size; const int nstripes = outer_size * inner_size / 1024; + size_t num_slices_per_batch = 1; + for (int d = batch_dims; d < static_cast(q) - 1; ++d) + num_slices_per_batch *= indices.size[d]; + parallel_for_(Range(0, outer_size), [&](const Range& range) { for (size_t i = range.start; i < range.end; ++i) { @@ -163,8 +167,10 @@ public: } if (batch_dims > 0) - offset += data_strides[batch_dims - 1] * i; - + { + const int batch_idx = i / num_slices_per_batch; + offset += (size_t)batch_idx * data_strides[batch_dims - 1]; + } // copy data from data to out for (size_t j = 0; j < inner_size; ++j) {