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

Unexpected warning output #1871

Closed
baentsch opened this issue Jul 27, 2024 · 0 comments · Fixed by #1874
Closed

Unexpected warning output #1871

baentsch opened this issue Jul 27, 2024 · 0 comments · Fixed by #1874
Milestone

Comments

@baentsch
Copy link
Member

Describe the bug
liboqs build should complete without error or warning output on supported platforms.

To Reproduce
run cmake -GNinja -B build -S . && cd build && ninja

Expected behavior
Correct build without error or warning output.

Screenshots
Actual output:

[1408/1442] Building C object tests/CMakeFiles/kat_sig_stfl.dir/kat_sig_stfl.c.o
/home/baentsch/git/oqs/liboqs/tests/kat_sig_stfl.c: In function ‘sig_stfl_kat’:
/home/baentsch/git/oqs/liboqs/tests/kat_sig_stfl.c:314:17: warning: ignoring return value of ‘fscanf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  314 |                 fscanf(fp_rsp, "%llu", &sigs_remain);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/baentsch/git/oqs/liboqs/tests/kat_sig_stfl.c:323:17: warning: ignoring return value of ‘fscanf’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
  323 |                 fscanf(fp_rsp, "%llu", &sigs_maximum);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[1442/1442] Linking C executable tests/dump_alg_info

Environment (please complete the following information):

  • OS: [Ubuntu 22]
  • OpenSSL version [3.0.2]
  • Compiler version used [gcc 11.4]
  • Build variables used [None -- see command above]
  • liboqs version [main branch]

Tagging @cothan @ashman-p as this might be related to missing STFL guards/#defines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant