Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FS#4000 - firewall3: Build failure with -Werror and gcc 10 #8997

Open
openwrt-bot opened this issue Sep 2, 2021 · 1 comment
Open

FS#4000 - firewall3: Build failure with -Werror and gcc 10 #8997

openwrt-bot opened this issue Sep 2, 2021 · 1 comment
Labels
core packages flyspray release/21.02

Comments

@openwrt-bot
Copy link

openwrt-bot commented Sep 2, 2021

dansan:

Occurs on today's openwrt-21.02 branch.

/home/daniel/proj/embedded/openwrt/v21.02.y/build_dir/target-mipsel_24kc_musl/firewall-2021-03-23-61db17ed/options.c: In function 'fw3_parse_options':
/home/daniel/proj/embedded/openwrt/v21.02.y/build_dir/target-mipsel_24kc_musl/firewall-2021-03-23-61db17ed/options.c:1113:7: error: '%s' directive argument is null [-Werror=format-overflow=]
1113 | warn_elem(e, "has invalid value '%s'", p);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

I'm behind on GCC development, but it looks like they're generating some interesting warnings now after constant propagation.

@openwrt-bot
Copy link
Author

openwrt-bot commented Sep 2, 2021

dansan:

Sorry, not familiar with Flyspray yet:

/home/daniel/proj/embedded/openwrt/v21.02.y/build_dir/target-mipsel_24kc_musl/firewall-2021-03-23-61db17ed/options.c: In function 'fw3_parse_options':
/home/daniel/proj/embedded/openwrt/v21.02.y/build_dir/target-mipsel_24kc_musl/firewall-2021-03-23-61db17ed/options.c:1113:7: error: '%s' directive argument is null [-Werror=format-overflow=]
1113 | warn_elem(e, "has invalid value '%s'", p);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

@aparcar aparcar added release/21.02 core packages labels Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core packages flyspray release/21.02
Projects
None yet
Development

No branches or pull requests

2 participants