RESOLVED FIXED 220869
Nullptr dereference in SpeechRecognition::stop()
https://bugs.webkit.org/show_bug.cgi?id=220869
Summary Nullptr dereference in SpeechRecognition::stop()
Sihui Liu
Reported 2021-01-22 11:02:01 PST
...
Attachments
Patch (1.66 KB, patch)
2021-01-22 11:19 PST, Sihui Liu
no flags
Patch (1.66 KB, patch)
2021-01-22 11:21 PST, Sihui Liu
no flags
Patch (3.71 KB, patch)
2021-01-22 18:35 PST, Sihui Liu
no flags
Patch for landing (3.71 KB, patch)
2021-01-24 17:24 PST, Sihui Liu
no flags
Radar WebKit Bug Importer
Comment 1 2021-01-22 11:02:12 PST Comment hidden (obsolete)
Sihui Liu
Comment 2 2021-01-22 11:19:29 PST
Sihui Liu
Comment 3 2021-01-22 11:20:40 PST
Sihui Liu
Comment 4 2021-01-22 11:21:07 PST
Ryosuke Niwa
Comment 5 2021-01-22 12:38:51 PST
Comment on attachment 418160 [details] Patch Is there a security issue? If not, can we add a test?
youenn fablet
Comment 6 2021-01-22 13:46:34 PST
It seems a test can be added if this is about frameless document.
Sihui Liu
Comment 7 2021-01-22 18:35:29 PST
Ryosuke Niwa
Comment 8 2021-01-22 22:05:38 PST
Comment on attachment 418205 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=418205&action=review > LayoutTests/fast/speechrecognition/detached-iframe.html:8 > + > + Two blank lines. > LayoutTests/fast/speechrecognition/detached-iframe.html:11 > +if (window.testRunner) { > + jsTestIsAsync = true; > +} No curly braces around a single line statement. > LayoutTests/fast/speechrecognition/detached-iframe.html:14 > + var frame = document.getElementById('iframe'); Use const?
Sihui Liu
Comment 9 2021-01-24 17:24:56 PST
Created attachment 418247 [details] Patch for landing
EWS
Comment 10 2021-01-24 17:59:58 PST
commit-queue failed to commit attachment 418247 [details] to WebKit repository.
EWS
Comment 11 2021-01-25 04:31:04 PST
Committed r271791: <https://trac.webkit.org/changeset/271791> All reviewed patches have been landed. Closing bug and clearing flags on attachment 418247 [details].
Note You need to log in before you can comment on or make changes to this bug.