RESOLVED FIXED 191892
[SOUP] Follow-up robustness improvements to the certificate decoder 
https://bugs.webkit.org/show_bug.cgi?id=191892
Summary [SOUP] Follow-up robustness improvements to the certificate decoder
Claudio Saavedra
Reported 2018-11-21 10:28:47 PST
[SOUP] Follow-up robustness improvements to the certificate decoder
Attachments
Patch (2.29 KB, patch)
2018-11-21 10:31 PST, Claudio Saavedra 		mcatanzaro: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Claudio Saavedra
Comment 1 2018-11-21 10:31:34 PST
Created attachment 355427 [details] Patch
Adrian Perez
Comment 2 2018-11-21 10:39:05 PST
Comment on attachment 355427 [details] Patch Patch LGTM, r+ (informal review, tho :)
Michael Catanzaro
Comment 3 2018-11-21 10:41:54 PST
Comment on attachment 355427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=355427&action=review Yes and yes, good catches. > Source/WebCore/ChangeLog:10 > + is nil, return false from the decoder to notify the failure null, what language are you working with here :P
Claudio Saavedra
Comment 4 2018-11-21 10:58:28 PST
Committed r238426: <https://trac.webkit.org/changeset/238426>
Note You need to log in before you can comment on or make changes to this bug.