New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 659210 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Chrome Remote Desktop is crashing my Pixel on remote session initiation

Project Member Reported by danan@google.com, Oct 25 2016

Issue description

IMPORTANT: Your crash has already been automatically reported to our crash system. Please file this bug only if you can provide more information about it.


Chrome Version: 54.0.2840.68
Operating System: Linux 3.14.0

URL (if applicable) where crash occurred:
Using chrome remote desktop

Can you reproduce this crash?
Yes

What steps will reproduce this crash? (If it's not reproducible, what were you doing just before the crash?)
1. Start a chrome-remote-desktop connection.
2.
3.

****DO NOT CHANGE BELOW THIS LINE****
Crash ID: crash/b63ee26b00000000

 
Components: Services>Chromoting
How are you using Chrome Remote Desktop? As a client (ie, connecting to another computer) or a host (ie, sharing your Chromebook with someone else)? At what point does it crash?

Offline, you indicated that this also affects desktop sharing in Hangouts; at what point does it crash in that scenario?
Owner: sergeyu@chromium.org
Status: Assigned (was: Unconfirmed)
Looking at the crash logs, it looks like this is new in M54, and affects both Chrome and ChromeOS.
Status: Fx (was: Assigned)
Looks like this has already been fixed in https://chromiumcodereview.appspot.com/2435033002
Status: Fixed (was: Fx)
Labels: -Restrict-View-EditIssue M-55 Merge-Request-55
Status: Started (was: Fixed)
Requesting merge to M55 branch as it affects not only chromoting, but all Aura-based screen/window capturing in Chrome, including Windows.
Components: Blink>GetUserMedia>Desktop

Comment 8 by dimu@chromium.org, Oct 27 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)
Project Member

Comment 9 by sheriffbot@chromium.org, Oct 31 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 10 by bugdroid1@chromium.org, Oct 31 2016

Labels: -merge-approved-55 merge-merged-2883
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/149b81a0e8d4ef9ea107489aa62b6ffc429cba21

commit 149b81a0e8d4ef9ea107489aa62b6ffc429cba21
Author: Sergey Ulanov <sergeyu@chromium.org>
Date: Mon Oct 31 18:36:53 2016

Fix NULL pointer dereference in FinishedAsyncCopyRequest()

Due to undefined (favorably right to left) argument evaluation order,
the tracker might have been passed and set to NULL before the window
is looked up which results in a NULL pointer dereference.

BUG= 659210 

Review-Url: https://chromiumcodereview.appspot.com/2435033002
Cr-Commit-Position: refs/heads/master@{#426748}
(cherry picked from commit 1b4c64279b7c212242738b96643a0700d04d725c)

Review URL: https://codereview.chromium.org/2465993002 .

Cr-Commit-Position: refs/branch-heads/2883@{#386}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/149b81a0e8d4ef9ea107489aa62b6ffc429cba21/ui/snapshot/snapshot_aura.cc

Labels: -merge-merged-2883 Merge-Approved-55
Status: Fixed (was: Started)
https://chromium.googlesource.com/chromium/src/+/149b81a0e8d4ef9ea107489aa62b6ffc429cba21
Project Member

Comment 12 by sheriffbot@chromium.org, Nov 3 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: jbau...@chromium.org
Labels: Merge-Request-54
Maybe we should try to get this fix into M54 as well.

Comment 14 by dimu@chromium.org, Nov 18 2016

Labels: -Merge-Request-54 Merge-Review-54 Hotlist-Merge-Review
[Automated comment] Request affecting a post-stable build (M54), manual review required.
No more M54s planned AFAIK.  If you can release out of band and you'd like to, ping me.
Labels: -Merge-Review-54

Sign in to add a comment