diff options
author | Ben Hilburn <ben.hilburn@ettus.com> | 2015-08-28 20:20:29 -0700 |
---|---|---|
committer | Ben Hilburn <ben.hilburn@ettus.com> | 2015-08-28 20:20:29 -0700 |
commit | ff5eef1c94649a26b893bbf45455190bc72c6b4a (patch) | |
tree | baf2603571f0f4805db67829ff91977d47fbfbc3 /gr-blocks/lib/file_meta_source_impl.cc | |
parent | b19cb7e746c5482a8890b1711d1984dd04c2bb38 (diff) |
Addresses Defects 1046385 & 1046340: Out-of-bounds access
Both of these defects are for the second parameter of the `constellation`
class' `map_to_points` function that accepts a pointer to an array of
`gr_complex` values. In both of these defects, a class is calling this function
and passing the address of a single `gr_complex` value in place of an array
pointer. The only reason this isn't exploding with SEGFAULTs is because both of
these functions happen to use the default constructor of `constellation`, which
sets the loop limit in `map_to_points` to `1`. It's generally a dangerous
design, but changing the function prototype seems heavy heanded, and adding
additional conditionals will end up affecting the fast-path. For now, I am just
documenting this oddity in the code.
Diffstat (limited to 'gr-blocks/lib/file_meta_source_impl.cc')
0 files changed, 0 insertions, 0 deletions