RESOLVED FIXED 47717
[GTK] Do a stricter check for invalid base64 dataURLs 
https://bugs.webkit.org/show_bug.cgi?id=47717
Summary [GTK] Do a stricter check for invalid base64 dataURLs
Sergio Villar Senin
Reported 2010-10-15 03:08:07 PDT
In https://bugs.webkit.org/show_bug.cgi?id=47666 we rolled back a more strict version of base64 decoding added in https://bugs.webkit.org/show_bug.cgi?id=44261 because the test LayoutTests/editing/undo/orphaned-selection-crash-bug32823-2.html started to fail. As commented here https://bugs.webkit.org/show_bug.cgi?id=47661#c10 the actual problem was not the test but the new code in ResourceHandleSoup.cpp that incorrectly does not finish the load of the dataURL if the base64 decoding fails. We should restore the old strict base64 decoding and gracefully end the load when an error occurs.
Attachments
Fix for the bug (3.14 KB, patch)
2010-10-15 03:19 PDT, Sergio Villar Senin 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sergio Villar Senin
Comment 1 2010-10-15 03:19:33 PDT
Created attachment 70846 [details] Fix for the bug This patch both restores the more strict base64 decoding and it also allows ResourceHandleSoup to properly finish the load of the resource when invalid base64 data is provided.
Sergio Villar Senin
Comment 2 2010-10-15 03:20:19 PDT
Ryosuke might be interested in the fix
Xan Lopez
Comment 3 2010-10-15 03:23:01 PDT
Comment on attachment 70846 [details] Fix for the bug Looks good to me.
WebKit Commit Bot
Comment 4 2010-10-15 03:39:16 PDT
Comment on attachment 70846 [details] Fix for the bug Clearing flags on attachment: 70846 Committed r69848: <http://trac.webkit.org/changeset/69848>
WebKit Commit Bot
Comment 5 2010-10-15 03:39:21 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.