[webkit-reviews] review granted: [Bug 184529] atob() should not accept a vertical tab : [Attachment 398219] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 1 18:42:53 PDT 2020

Darin Adler <darin at apple.com> has granted Rob Buis <rbuis at igalia.com>'s request
for review:
Bug 184529: atob() should not accept a vertical tab

Attachment 398219: Patch


--- Comment #2 from Darin Adler <darin at apple.com> ---
Comment on attachment 398219
  --> https://bugs.webkit.org/attachment.cgi?id=398219

View in context: https://bugs.webkit.org/attachment.cgi?id=398219&action=review

This change is OK, but I think it’s possibly needed because of a mistake.

> Source/WTF/wtf/text/Base64.cpp:213
> +		   || (!isSpaceOrNewline(ch) || ((options &
Base64DiscardVerticalTab) && ch == '\v'))) {

I don’t understand why we ever use the full-Unicode isSpaceOrNewline function.
In what context do we need to allow non-ASCII spaces? Do we have cross-browser
tests checking this behavior? What spaces exactly do we need to ignore? Does
any caller need ignore vertical tabs?

More information about the webkit-reviews mailing list