diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-21 14:12:09 +0200 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2018-10-21 20:14:18 +0200 |
| commit | c60e737832fe5f9b4841ba163e22c532f0abe6f4 (patch) | |
| tree | 58ff8c06f083c7435f2bccb42c0d981d19b858cc /svx | |
| parent | aacbb8bac09409d5a7d31dd8278dd495a0557a49 (diff) | |
tdf#120703 (PVS): redundant nullptr check
V668 There is no sense in testing the 'pDest' pointer against null, as the
memory was allocated using the 'new' operator. The exception will be
generated in the case of memory allocation error.
Change-Id: I17773f92ed73359e925ad779d171ec99727f7905
Reviewed-on: https://gerrit.libreoffice.org/62133
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/xml/xmlgrhlp.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/xml/xmlgrhlp.cxx b/svx/source/xml/xmlgrhlp.cxx index b45954fdbd0d..ca23ea3dc7a2 100644 --- a/svx/source/xml/xmlgrhlp.cxx +++ b/svx/source/xml/xmlgrhlp.cxx @@ -331,7 +331,7 @@ Graphic SvXMLGraphicOutputStream::GetGraphic() mpOStm->Seek( 0 ); aZCodec.Decompress( *mpOStm, *pDest ); - if (aZCodec.EndCompression() && pDest ) + if (aZCodec.EndCompression()) { sal_uIntPtr nStreamLen_ = pDest->TellEnd(); if (nStreamLen_) |