diff options
Diffstat (limited to 'svl')
-rw-r--r-- | svl/source/items/whiter.cxx | 10 | ||||
-rw-r--r-- | svl/source/misc/strmadpt.cxx | 1 |
2 files changed, 3 insertions, 8 deletions
diff --git a/svl/source/items/whiter.cxx b/svl/source/items/whiter.cxx index 9a4e748a62ad..d69960c6f628 100644 --- a/svl/source/items/whiter.cxx +++ b/svl/source/items/whiter.cxx @@ -26,18 +26,14 @@ SfxWhichIter::SfxWhichIter( const SfxItemSet& rSet, sal_uInt16 nFromWh, sal_uInt pStart(rSet.GetRanges()), nOfst(0), nFrom(nFromWh), nTo(nToWh) { - if ( nFrom > 0 ) - FirstWhich(); + if (nFrom > 0) + (void)FirstWhich(); } - - SfxWhichIter::~SfxWhichIter() { } - - sal_uInt16 SfxWhichIter::NextWhich() { while ( 0 != *pRanges ) @@ -56,8 +52,6 @@ sal_uInt16 SfxWhichIter::NextWhich() return 0; } - - sal_uInt16 SfxWhichIter::FirstWhich() { pRanges = pStart; diff --git a/svl/source/misc/strmadpt.cxx b/svl/source/misc/strmadpt.cxx index 7b8dc4e05168..207e52137668 100644 --- a/svl/source/misc/strmadpt.cxx +++ b/svl/source/misc/strmadpt.cxx @@ -945,6 +945,7 @@ bool SvDataPipe_Impl::removeMark(sal_uInt32 nPosition) if (t == m_aMarks.end()) return false; m_aMarks.erase(t); + // coverity[pass_freed_arg] - remove frees m_pFirstPage but then sets m_pFirstPage to something else while (remove(m_pFirstPage)) ; return true; } |