From a7e51326648b98b7dcb93819ff2df99c903c1214 Mon Sep 17 00:00:00 2001
From: Carles Fernandez <carles.fernandez@gmail.com>
Date: Fri, 14 Aug 2020 01:07:03 +0200
Subject: [PATCH] Fix bug in acquisition detection when the configuration
 parameter Acquisition_XX.threshold was set but Acquisition_XX.pfa was not

---
 docs/changelog.md                                           | 6 ++++++
 .../acquisition/gnuradio_blocks/pcps_acquisition.cc         | 2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/docs/changelog.md b/docs/changelog.md
index 95aede92a..608354bb2 100644
--- a/docs/changelog.md
+++ b/docs/changelog.md
@@ -10,6 +10,12 @@ SPDX-FileCopyrightText: 2011-2020 Carles Fernandez-Prades <carles.fernandez@cttc
 
 ## [Unreleased](https://github.com/gnss-sdr/gnss-sdr/tree/next)
 
+### Improvements in Availability:
+
+- Fixed bug in acquisition detection when the configuration parameter
+  `Acquisition_XX.threshold` was set but `Acquisition_XX.pfa` was not, causing
+  false locks.
+
 ### Improvements in Efficiency:
 
 - Faster `SignalConditioner` block when its implementation is set to
diff --git a/src/algorithms/acquisition/gnuradio_blocks/pcps_acquisition.cc b/src/algorithms/acquisition/gnuradio_blocks/pcps_acquisition.cc
index 905207646..c53b363f4 100644
--- a/src/algorithms/acquisition/gnuradio_blocks/pcps_acquisition.cc
+++ b/src/algorithms/acquisition/gnuradio_blocks/pcps_acquisition.cc
@@ -592,7 +592,7 @@ float pcps_acquisition::first_vs_second_peak_statistic(uint32_t& indext, int32_t
         }
 
     int32_t idx = excludeRangeIndex1;
-    memcpy(d_tmp_buffer.data(), d_magnitude_grid[index_doppler].data(), d_fft_size);
+    memcpy(d_tmp_buffer.data(), d_magnitude_grid[index_doppler].data(), d_fft_size * sizeof(float));
     do
         {
             d_tmp_buffer[idx] = 0.0;