Change in ovirt-engine[master]: userportal: closing RDP crashes IE

tjelinek at redhat.com tjelinek at redhat.com
Fri Apr 26 11:15:12 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: userportal: closing RDP crashes IE
......................................................................


userportal: closing RDP crashes IE

The problem was, that the MsRdpClient object has been removed from the body in
the "OnDisconnected" event of the MsRdpClient which is too soon and causes the
IE to crash - possibly because IE starts to destroy this object but after
OnDisconnected callback returns, IE tries to give the control back to the
MsRdpClient.

Solved by just removing this line becaue the proper cleanup is already done in
connect() method which removes the previous MsRdpClient if there is one.

Change-Id: I6501bbd88ed77df2d3f8acb7fe61913a2888ce7f
Bug-Url: https://bugzilla.redhat.com/909109
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/RdpConnector.java
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/14223
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6501bbd88ed77df2d3f8acb7fe61913a2888ce7f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list