diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-04-16 20:41:44 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2014-04-22 09:41:19 +0200 |
commit | f2a58c8d470f027b1e3ece0b26da804a4ab169dd (patch) | |
tree | 35871e05a85a1c3008bbf521d57cc7ee222eb517 /sd | |
parent | 544fbfeb5dc05d975337722950b8b059b2c107bd (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 ); } |