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

0.15.1: build fails with implicit declaration of function ‘s2n_cleanup_final’ #688

Closed
1 task
kloczek opened this issue Oct 31, 2024 · 4 comments
Closed
1 task
Labels
bug This issue is a bug. dependencies This issue is a problem in a dependency. p1 This is a high priority issue pending-release This issue will be fixed by an approved PR that hasn't been released yet.

Comments

@kloczek
Copy link

kloczek commented Oct 31, 2024

Describe the bug

Build fails with implicit declaration of function ‘s2n_cleanup_final’.

Regression Issue

  • Select this option if this issue appears to be a regression.

Expected Behavior

Build shuld not fail.

Current Behavior

/usr/bin/gcc -DAWS_CAL_USE_IMPORT_EXPORT -DAWS_COMMON_USE_IMPORT_EXPORT -DAWS_IO_EXPORTS -DAWS_IO_USE_IMPORT_EXPORT -DAWS_USE_EPOLL -DHAVE_SYSCONF -DINTEL_NO_ITTNOTIFY_API -DUSE_S2N -Daws_c
_io_EXPORTS -I/home/tkloczko/rpmbuild/BUILD/aws-c-io-0.15.1/include -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -O2 -g -grecord-gcc-switches -p
ipe -mtls-dialect=gnu2 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fdata-sections -ffunction-sections -fstack-protector-strong -specs
=/usr/lib/rpm/redhat/redhat-hardened-cc1 -flto=auto -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -Wall -Werror=format-security -O2 -g -DNDEBUG -std=gnu99 -fPIC -fvisi
bility=hidden -Wall -Wstrict-prototypes -fno-omit-frame-pointer -Wextra -pedantic -Wno-long-long -fPIC -MD -MT CMakeFiles/aws-c-io.dir/source/tls_channel_handler.c.o -MF CMakeFiles/aws-c-io
.dir/source/tls_channel_handler.c.o.d -o CMakeFiles/aws-c-io.dir/source/tls_channel_handler.c.o -c /home/tkloczko/rpmbuild/BUILD/aws-c-io-0.15.1/source/tls_channel_handler.c
/home/tkloczko/rpmbuild/BUILD/aws-c-io-0.15.1/source/s2n/s2n_tls_channel_handler.c: In function ‘aws_tls_clean_up_static_state’:
/home/tkloczko/rpmbuild/BUILD/aws-c-io-0.15.1/source/s2n/s2n_tls_channel_handler.c:258:9: error: implicit declaration of function ‘s2n_cleanup_final’; did you mean ‘s2n_cleanup_thread’? [-W
implicit-function-declaration]
  258 |         s2n_cleanup_final();
      |         ^~~~~~~~~~~~~~~~~
      |         s2n_cleanup_thread
gmake[2]: *** [CMakeFiles/aws-c-io.dir/build.make:345: CMakeFiles/aws-c-io.dir/source/s2n/s2n_tls_channel_handler.c.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....

s2n-tls 1.5.6.

Reproduction Steps

Try to build with s2n-tls 1.5.6.

Possible Solution

No response

Additional Information/Context

No response

aws-c-io version used

0.15.1

Compiler and version used

gcc 14

Operating System and version

Linux x86/64

Tasks

Preview Give feedback
No tasks being tracked yet.
@kloczek kloczek added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Oct 31, 2024
@graebm
Copy link
Contributor

graebm commented Oct 31, 2024

s2n_cleanup_final was added recently to s2n-tls very recently: aws/s2n-tls@9819ac0. You can use that specific commit of s2n-tls, or their main branch, until they make another release (likely within a week). Or you can hold back updating aws-c-io to

Sorry, this was a case of bad timing. The s2n-tls team asked us to use this function, and we didn't realize it wasn't available yet in a tagged release (aws-c-io's tests build against the main branch of s2n-tls). I've asked the s2n-tls team to do their next release ASAP, but I don't know what rigamarole they might go through before each release.

I'll post more when I have updates...

@jmklix jmklix added dependencies This issue is a problem in a dependency. pending-release This issue will be fixed by an approved PR that hasn't been released yet. p1 This is a high priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Oct 31, 2024
@kloczek
Copy link
Author

kloczek commented Oct 31, 2024

Don't worry about me. I can wait until next release 😄
Thank you for confirmation that it is real issue 👍

@graebm
Copy link
Contributor

graebm commented Nov 1, 2024

Ok, fixed if you use the latest S2N-TLS release: v1.5.7

@graebm graebm closed this as completed Nov 1, 2024
@kloczek
Copy link
Author

kloczek commented Nov 1, 2024

Just tested and all looks good now 👍
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. dependencies This issue is a problem in a dependency. p1 This is a high priority issue pending-release This issue will be fixed by an approved PR that hasn't been released yet.
Projects
None yet
Development

No branches or pull requests

3 participants