WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
Formatted Diff
Diff
Patch
(1.66 KB, patch)
2021-01-22 11:21 PST
,
Sihui Liu
no flags
Details
Formatted Diff
Diff
Patch
(3.71 KB, patch)
2021-01-22 18:35 PST
,
Sihui Liu
no flags
Details
Formatted Diff
Diff
Patch for landing
(3.71 KB, patch)
2021-01-24 17:24 PST
,
Sihui Liu
no flags
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2021-01-22 11:02:12 PST
Comment hidden (obsolete)
<
rdar://problem/73507085
>
Sihui Liu
Comment 2
2021-01-22 11:19:29 PST
Created
attachment 418159
[details]
Patch
Sihui Liu
Comment 3
2021-01-22 11:20:40 PST
<
rdar://problem/73403469
>
Sihui Liu
Comment 4
2021-01-22 11:21:07 PST
Created
attachment 418160
[details]
Patch
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
Created
attachment 418205
[details]
Patch
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.
Top of Page
Format For Printing
XML
Clone This Bug