Skip to content

Commit

Permalink
Fixed MonitorData deletion in Monitor2Shutdown.
Browse files Browse the repository at this point in the history
Removed temp variables to avoid other confusions.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284812 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
vitalybuka@chromium.org committed Jul 22, 2014
1 parent 67ca3c6 commit cf700d7
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions cloud_print/virtual_driver/win/port_monitor/port_monitor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -515,16 +515,13 @@ BOOL WINAPI Monitor2ClosePort(HANDLE port_handle) {
SetLastError(ERROR_INVALID_PARAMETER);
return FALSE;
}
PortData* port_data = reinterpret_cast<PortData*>(port_handle);
delete port_data;
delete reinterpret_cast<PortData*>(port_handle);
return TRUE;
}

VOID WINAPI Monitor2Shutdown(HANDLE monitor_handle) {
if (monitor_handle != NULL) {
MonitorData* monitor_data =
reinterpret_cast<MonitorData*>(monitor_handle);
delete monitor_handle;
delete reinterpret_cast<MonitorData*>(monitor_handle);
}
}

Expand Down Expand Up @@ -581,8 +578,7 @@ DWORD WINAPI Monitor2XcvDataPort(HANDLE xcv_handle,
}

BOOL WINAPI Monitor2XcvClosePort(HANDLE handle) {
XcvUiData* xcv_data = reinterpret_cast<XcvUiData*>(handle);
delete xcv_data;
delete reinterpret_cast<XcvUiData*>(handle);
return TRUE;
}

Expand Down

0 comments on commit cf700d7

Please sign in to comment.