diff options
| author | Marcus Müller <marcus.mueller@ettus.com> | 2018-07-08 14:55:04 +0200 |
|---|---|---|
| committer | Marcus Müller <marcus@hostalia.de> | 2018-07-12 20:35:33 +0200 |
| commit | 0f23efbaddc559aba99cf6c33f527a99fcea785e (patch) | |
| tree | f8ff362645127219bd382680d0a416a6e71d4ddd /gr-fec/lib | |
| parent | 84193bf1b00def9789e4aff16edaa030de17a0d7 (diff) | |
Fix vector out of bound accesses
Unless most of the other #1776-related fixes, this actually was an
effect of reading from unitialized memory. I'm surprised this worked as
intended!
Needs content-wise review from @jdemel!
Diffstat (limited to 'gr-fec/lib')
| -rw-r--r-- | gr-fec/lib/polar_common.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/gr-fec/lib/polar_common.cc b/gr-fec/lib/polar_common.cc index c04ec5a15c..fcb34c2d35 100644 --- a/gr-fec/lib/polar_common.cc +++ b/gr-fec/lib/polar_common.cc @@ -108,14 +108,13 @@ namespace gr { d_volk_temp = (unsigned char*) volk_malloc(sizeof(unsigned char) * block_size(), volk_get_alignment()); d_volk_frozen_bit_mask = (unsigned char*) volk_malloc(sizeof(unsigned char) * block_size(), volk_get_alignment()); d_volk_frozen_bits = (unsigned char*) volk_malloc(sizeof(unsigned char) * nfrozen, volk_get_alignment()); - for(int i = 0; i < nfrozen; i++){ - d_volk_frozen_bits[i] = d_frozen_bit_values[i]; - } + std::copy(d_frozen_bit_values.begin(), d_frozen_bit_values.end(), d_volk_frozen_bits); + std::fill(d_volk_frozen_bits + d_frozen_bit_values.size(), d_volk_frozen_bits + nfrozen, 0); int nfbit = 0; for(int i = 0; i < block_size(); i++){ unsigned char m = 0x00; - if(d_frozen_bit_positions[nfbit] == i){ + if(nfbit < d_frozen_bit_positions.size() && d_frozen_bit_positions[nfbit] == i){ m = 0xFF; nfbit++; } |