Skip to content

Fix format security warning in newer GCC#161

Merged
michalvasko merged 1 commit intoCESNET:masterfrom
neheb:f
Jul 16, 2019
Merged

Fix format security warning in newer GCC#161
michalvasko merged 1 commit intoCESNET:masterfrom
neheb:f

Conversation

@neheb
Copy link
Copy Markdown
Contributor

@neheb neheb commented Jul 16, 2019

In OpenWrt, -Werror=format-string is passed, making this fail compilation.

printf style functions are meant to work with format strings anyway.

In OpenWrt, -Werror=format-string is passed, making this fail compilation.

printf style functions are meant to work with format strings anyway.
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #161 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
- Coverage   38.14%   38.12%   -0.03%     
==========================================
  Files          23       23              
  Lines        9959     9959              
  Branches     1948     1948              
==========================================
- Hits         3799     3797       -2     
- Misses       5216     5217       +1     
- Partials      944      945       +1
Impacted Files Coverage Δ
src/io.c 45.47% <0%> (ø) ⬆️
src/session_server_ssh.c 24.31% <0%> (-0.13%) ⬇️
src/session_server_tls.c 28.24% <0%> (-0.11%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7879bf3...194854b. Read the comment docs.

@michalvasko
Copy link
Copy Markdown
Member

Hi,
okay, no problem.

Regards,
Michal

@michalvasko michalvasko merged commit cd8e449 into CESNET:master Jul 16, 2019
@neheb neheb deleted the f branch July 16, 2019 19:36
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.

3 participants