Skip to content

Fix compile error with gcc14#984

Open
MartinHerren wants to merge 1 commit intoNuand:masterfrom
MartinHerren:fix_gcc14_compile_error
Open

Fix compile error with gcc14#984
MartinHerren wants to merge 1 commit intoNuand:masterfrom
MartinHerren:fix_gcc14_compile_error

Conversation

@MartinHerren
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

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