diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2014-04-16 20:41:44 +0100 |
|---|---|---|
| committer | Andrzej Hunt <andrzej.hunt@collabora.com> | 2014-04-18 10:57:56 +0000 |
| commit | bd10baa8565ac4ea8697a4e9ed8c5a489ae7a4fb (patch) | |
| tree | 2ff0bfe0720fb09b2b2ee1ca1d9cd80c0925f058 /sd | |
| parent | 0826bf264c7308be1da045b79bdcc2cff8b7cb42 (diff) | |
coverity#1202882 Dereference before null check
Change-Id: Ie61df08c11b687c0b4a3ae212b3f9a6c95171396
(cherry picked from commit c152349f69acfb6bb873ed1cd12ca1b6dc325f1f)
Reviewed-on: https://gerrit.libreoffice.org/9095
Tested-by: Andrzej Hunt <andrzej.hunt@collabora.com>
Reviewed-by: Andrzej Hunt <andrzej.hunt@collabora.com>
Diffstat (limited to 'sd')
| -rw-r--r-- | sd/source/ui/remotecontrol/Communicator.cxx | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sd/source/ui/remotecontrol/Communicator.cxx b/sd/source/ui/remotecontrol/Communicator.cxx index d3af69715190..bf399cae8ceb 100644 --- a/sd/source/ui/remotecontrol/Communicator.cxx +++ b/sd/source/ui/remotecontrol/Communicator.cxx @@ -122,10 +122,9 @@ void Communicator::execute() pTransmitter->join(); pTransmitter = NULL; - if( mpSocket ) - mpSocket->close(); + mpSocket->close(); delete mpSocket; - + mpSocket = NULL; RemoteServer::removeCommunicator( this ); } |