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

build warnings core cfg #3208

Closed
sergey-safarov opened this issue Aug 8, 2022 · 1 comment
Closed

build warnings core cfg #3208

sergey-safarov opened this issue Aug 8, 2022 · 1 comment

Comments

@sergey-safarov
Copy link
Member

CC (gcc) [kamailio]		core/crypto/shautils.o
CC (gcc) [kamailio]		core/cfg/cfg.o
CC (gcc) [kamailio]		core/cfg/cfg_ctx.o
core/cfg/cfg_ctx.c: In function 'cfg_update_defaults':
core/cfg/cfg_ctx.c:287:28: warning: the comparison will always evaluate as 'true' for the pointer operand in '(unsigned char *)&ginst->vars + (sizetype)var->offset' must not be NULL [-Waddress]
  287 |                         if(ginst->vars + var->offset) {
      |                            ^~~~~
CC (gcc) [kamailio]		core/cfg/cfg_script.o
CC (gcc) [kamailio]		core/cfg/cfg_select.o
@henningw henningw changed the title build warnings build warnings core cfg Aug 8, 2022
@miconda
Copy link
Member

miconda commented Aug 12, 2022

Not sure about the purpose of the check, not being the developer, probably is useless, but I pushed a commit to silent the warning just to play safe for the moment.

@miconda miconda closed this as completed Aug 12, 2022
miconda added a commit that referenced this issue Sep 14, 2022
miconda added a commit that referenced this issue Sep 15, 2022
- GH #3208

(cherry picked from commit 26785f8)
(cherry picked from commit 8aceadf)
miconda added a commit that referenced this issue Sep 15, 2022
- GH #3208

(cherry picked from commit 26785f8)
(cherry picked from commit 8aceadf)
(cherry picked from commit e90c246)
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

No branches or pull requests

2 participants