[//gpu] Fix dangling pointers in GpuServiceImpl destruction [chromium/src : main]
0 views
Skip to first unread message
Vasiliy Telezhnikov (Gerrit)
Sep 25, 2025, 11:29:56 AM (15 hours ago) Sep 25
to Colin Blundell, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Colin Blundell
Vasiliy Telezhnikov voted and added 1 comment![]( "Open in Gerrit")
Votes added by Vasiliy Telezhnikov
| Code-Review | +1 |
1 comment
Patchset-level comments
Related details
Attention is currently required from:
- Colin Blundell
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Colin Blundell (Gerrit)
Sep 25, 2025, 11:34:07 AM (15 hours ago) Sep 25
to Colin Blundell, Vasiliy Telezhnikov, Chromium LUCI CQ, chromium...@chromium.org
Colin Blundell voted and added 1 comment![]( "Open in Gerrit")
Votes added by Colin Blundell
| Commit-Queue | +2 |
1 comment
Patchset-level comments
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Sep 25, 2025, 12:08:47 PM (15 hours ago) Sep 25
to Colin Blundell, Vasiliy Telezhnikov, chromium...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[//gpu] Fix dangling pointers in GpuServiceImpl destruction
Issue that was uncovered by a recent unrelated refactoring; see analysis
in [1] (it turned out that there was the same issue with
`GpuServiceImpl::owned_scheduler_`; this CL fixes both).
[1]: https://issues.chromium.org/u/1/issues/447037086#comment4
Bug: 447037086
Change-Id: I1fc02cfed8810ed3f86edd17ef0fee7bceb2f36f
Reviewed-by: Vasiliy Telezhnikov <vas...@chromium.org>
Commit-Queue: Colin Blundell <blun...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1520656}
Files:
- M gpu/ipc/service/gpu_channel.cc
- M gpu/ipc/service/image_decode_accelerator_stub.cc
- M gpu/ipc/service/image_decode_accelerator_stub.h
Change size: S
Delta: 3 files changed, 8 insertions(+), 2 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Vasiliy Telezhnikov
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages