RESOLVED FIXED 234332
[GTK][WPE] Build fix for Debian Stable after r287015
https://bugs.webkit.org/show_bug.cgi?id=234332
Summary [GTK][WPE] Build fix for Debian Stable after r287015
Diego Pino
Reported 2021-12-14 19:39:47 PST
[GTK][WPE] Build fix for Debian Stable after r287015
Attachments
Patch (1.96 KB, patch)
2021-12-14 19:40 PST, Diego Pino
no flags
Patch (1.93 KB, patch)
2021-12-14 19:46 PST, Diego Pino
no flags
Diego Pino
Comment 1 2021-12-14 19:40:12 PST
Diego Pino
Comment 2 2021-12-14 19:44:50 PST
Debian Stable build bot started failing r287015: https://build.webkit.org/#/builders/46 Compilation error was: ``` In file included from WebCore/DerivedSources/unified-sources/UnifiedSource-cbdfe323-30.cpp:8: ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp: In constructor ‘WebCore::PushCrypto::decryptAES128GCMPayload(const WebCore::PushCrypto::ClientKeys&, WTF::Span<const unsigned char, 18446744073709551615>)::KeyInfo::KeyInfo()’: ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp:129:12: error: char-array initialized from wide string struct KeyInfo { ^~~~~~~ ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp: In function ‘std::optional<WTF::Vector<unsigned char> > WebCore::PushCrypto::decryptAES128GCMPayload(const WebCore::PushCrypto::ClientKeys&, WTF::Span<const unsigned char, 18446744073709551615>)’: ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp:137:13: note: synthesized method ‘WebCore::PushCrypto::decryptAES128GCMPayload(const WebCore::PushCrypto::ClientKeys&, WTF::Span<const unsigned char, 18446744073709551615>)::KeyInfo::KeyInfo()’ first required here KeyInfo keyInfo; ^~~~~~~ ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp: In constructor ‘WebCore::PushCrypto::decryptAESGCMPayload(const WebCore::PushCrypto::ClientKeys&, WTF::Span<const unsigned char, 18446744073709551615>, WTF::Span<const unsigned char, 18446744073709551615>, WTF::Span<const unsigned char, 18446744073709551615>)::KeyDerivationContext::KeyDerivationContext()’: ../../Source/WebCore/Modules/push-api/PushMessageCrypto.cpp:253:12: error: char-array initialized from wide string struct KeyDerivationContext { ^~~~~~~~~~~~~~~~~~~~ ``` The bot uses GCC 8.3: ``` $ gcc --version gcc (Debian 8.3.0-6) 8.3.0 Copyright (C) 2018 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ```
Diego Pino
Comment 3 2021-12-14 19:46:31 PST
Ben Nham
Comment 4 2021-12-14 21:35:22 PST
Seems fine to me but I'm not a reviewer. I'll ping someone on the team to give the r+.
EWS
Comment 5 2021-12-15 01:10:47 PST
Committed r287065 (245260@main): <https://commits.webkit.org/245260@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 447193 [details].
Radar WebKit Bug Importer
Comment 6 2021-12-15 01:11:17 PST
Note You need to log in before you can comment on or make changes to this bug.