Skip to content

Conversation

@MartinHerren
Copy link
Contributor

This fixes a compile error with gcc14:
.../host/utilities/bladeRF-fsk/c/src/fir_filter.c:227:28: error: ‘calloc’ sizes specified with ‘sizeof’ in the earlier argument and not in the later argument [-Werror=calloc-transposed-args]
227 | outbuf = calloc(sizeof(struct complex_sample), chunk_size);

because the arguments of some callocs are inverted..

This fixes !972

This fixes a compile error with gcc14:
.../host/utilities/bladeRF-fsk/c/src/fir_filter.c:227:28: error: ‘calloc’ sizes specified with ‘sizeof’ in the earlier argument and not in the later argument [-Werror=calloc-transposed-args]
  227 |     outbuf = calloc(sizeof(struct complex_sample), chunk_size);

because the arguments of some callocs are inverted.

Signed-off-by: Martin Herren (HB9FXX) <cq@hb9fxx.ch>
@CLAassistant
Copy link

CLAassistant commented Sep 9, 2024

CLA assistant check
All committers have signed the CLA.

@paparodeo paparodeo mentioned this pull request Dec 6, 2024
FoxRomeo added a commit to FoxRomeo/dump1090-fa that referenced this pull request Feb 12, 2025
FoxRomeo added a commit to FoxRomeo/dump1090-fa that referenced this pull request Feb 12, 2025
@cyclic-pentane
Copy link

This PR and the corresponding can be closed now, since the fix has already been applied: https://github.com/Nuand/bladeRF-fsk/blob/fb903684036217b9db9dcd6be8007548e12a377e/c/src/fir_filter.c#L242

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Argument Order in calloc Call in fir_filter.c

3 participants