Skip to content

libwebsockets: Fix -Werror=enum-int-mismatch in lws_tls_server_abort_connection() - #21206

Merged
neheb merged 1 commit into
openwrt:masterfrom
oskarirauta:libwebsockets
Jul 1, 2023
Merged

libwebsockets: Fix -Werror=enum-int-mismatch in lws_tls_server_abort_connection()#21206
neheb merged 1 commit into
openwrt:masterfrom
oskarirauta:libwebsockets

Conversation

@oskarirauta

Copy link
Copy Markdown
Contributor

Maintainer: @karlp
Compile tested: x86_64, latest git
Run tested: x86_64, latest git

Description:

with gcc 13 libwebsockets fails with -Werror=enum-int-mismatch with both SSL variants. These 2 patches work out that issue. Problem is mentioned as a issue in #20949 and instead of overriding issue with appropriate CFLAGS, this approach was suggested for me as it's been made as a PR (for openssl) at warmcat/libwebsockets#2824

Second patch for mbedtls, was made by me using same approach as was used of openssl.

Eventually these propably will be merged into libwebsockets mainstream.

…connection()

with gcc 13 libwebsockets fails with -Werror=enum-int-mismatch with both
SSL variants. These 2 patches work out that issue. Problem is mentioned
as a issue in openwrt#20949 and instead of overriding issue
with appropriate CFLAGS, this approach was suggested for me as it's
been made as a PR (for openssl) at warmcat/libwebsockets#2824

Second patch for mbedtls, was made by me using same approach
as was used of openssl.

Eventually these propably will be merged into libwebsockets mainstream.

Signed-off-by: Oskari Rauta <oskari.rauta@gmail.com>
@oskarirauta

Copy link
Copy Markdown
Contributor Author

@karlp

ping

@BKPepe
BKPepe requested a review from karlp June 18, 2023 06:24
@neheb
neheb merged commit 6a897e2 into openwrt:master Jul 1, 2023
@oskarirauta
oskarirauta deleted the libwebsockets branch October 1, 2023 16:15
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.

2 participants