RESOLVED FIXED 252725
yasm: fix deprecation warnings for sprintf() 
https://bugs.webkit.org/show_bug.cgi?id=252725
Summary yasm: fix deprecation warnings for sprintf()
David Kilzer (:ddkilzer)
Reported 2023-02-21 22:17:27 PST
yasm: fix deprecation warnings for sprint() This is now causing internal build failures. Source/ThirdParty/libwebrtc/Source/third_party/yasm/frontends/yasm/yasm-options.c:177:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only. Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations] Source/ThirdParty/libwebrtc/Source/third_party/yasm/frontends/yasm/yasm-options.c:185:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only. Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations] Source/ThirdParty/libwebrtc/Source/third_party/yasm/frontends/yasm/yasm-options.c:193:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only. Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations] Source/ThirdParty/libwebrtc/Source/third_party/yasm/frontends/yasm/yasm-options.c:199:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only. Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations] Source/ThirdParty/libwebrtc/Source/third_party/yasm/modules/arch/x86/x86arch.c:169:17: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only. Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations]
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-02-21 22:17:48 PST
<rdar://problem/105763698>
David Kilzer (:ddkilzer)
Comment 2 2023-02-21 22:21:45 PST
Pull request: https://github.com/WebKit/WebKit/pull/10496
EWS
Comment 3 2023-02-21 22:38:55 PST
Committed 260657@main (d8e149964798): <https://commits.webkit.org/260657@main> Reviewed commits have been landed. Closing PR #10496 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.