281919 2024-10-22 14:22:56 -0700 WebCore::DecodedHTMLEntity::span() should use std::span::first() 2024-10-22 19:40:05 -0700 1 1 1 Unclassified WebKit WebCore Misc. Other Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=281892 https://bugs.webkit.org/show_bug.cgi?id=281917 InRadar P2 Normal --- 1 ddkilzer ddkilzer webkit-bug-importer oldest_to_newest 2069872 0 ddkilzer 2024-10-22 14:22:56 -0700 WebCore::DecodedHTMLEntity::span() should use std::span::first(). This code currently avoids bounds checking of `m_characters` when creating std::span: ``` constexpr std::span<const UChar> span() const { return { m_characters.data(), m_length }; } ``` Found by std::span clang static analysis checker under development. 2069876 1 webkit-bug-importer 2024-10-22 14:25:53 -0700 <rdar://problem/138431470> 2069885 2 ddkilzer 2024-10-22 14:32:15 -0700 Pull request: https://github.com/WebKit/WebKit/pull/35585 2069969 3 ews-feeder 2024-10-22 19:40:04 -0700 Committed 285583@main (b8b9ecf63b67): <https://commits.webkit.org/285583@main> Reviewed commits have been landed. Closing PR #35585 and removing active labels.