configmgr/source/valueparser.cxx | 1 -
cpputools/source/unoexe/unoexe.cxx | 1 -
dbaccess/source/filter/hsqldb/utils.cxx | 2 +-
desktop/source/deployment/gui/dp_gui_extlistbox.cxx | 1 -
desktop/source/deployment/registry/configuration/dp_configuration.cxx | 1 -
desktop/unx/source/start.c | 1 -
drawinglayer/source/primitive2d/textbreakuphelper.cxx | 1 -
editeng/source/editeng/impedit.cxx | 2 --
editeng/source/editeng/impedit.hxx | 4
----
embeddedobj/source/msole/ownview.cxx | 1 -
extensions/source/update/check/updatecheck.cxx | 2 --
filter/source/msfilter/mscodec.cxx | 1 -
filter/source/svg/svgexport.cxx | 1 -
filter/source/svg/svgwriter.cxx | 1 -
formula/source/core/api/FormulaCompiler.cxx | 1 -
fpicker/source/office/OfficeFilePicker.cxx | 1 -
framework/source/accelerators/acceleratorconfiguration.cxx | 2 --
oox/source/export/ColorPropertySet.cxx | 1 -
sc/source/core/data/column3.cxx | 1 -
sc/source/ui/view/tabview.cxx | 4
----
scaddins/source/analysis/analysis.cxx | 1 -
sd/source/ui/remotecontrol/BufferedStreamSocket.cxx | 3
---
svtools/source/brwbox/datwin.cxx | 1 -
svx/source/stbctrls/zoomsliderctrl.cxx | 1 -
sw/inc/view.hxx | 2 --
sw/source/core/view/pagepreviewlayout.cxx | 1 -
sw/source/filter/ww8/writerwordglue.cxx | 1 -
tools/source/datetime/tdate.cxx | 1 -
vcl/source/app/svcache.cxx | 1 -
vcl/source/outdev/map.cxx | 2 --
vcl/source/window/menu.cxx | 1 -
writerperfect/source/common/WPXSvInputStream.cxx | 1 -
32 files changed, 1 insertion(+), 45 deletions(-)
New commits:
commit 72e538b9bdcfc193ccef8c134bd3f4d79bdcc14c
Author: Caolán McNamara <caolan.mcnam...@collabora.com>
AuthorDate: Sat Aug 16 14:46:10 2025 +0100
Commit: Caolán McNamara <caolan.mcnam...@collabora.com>
CommitDate: Sun Aug 17 15:33:16 2025 +0200
tainted_data_return is deprecated
in favour of taint_source, lets drop them all first
Change-Id: I9c4e5c143fcbd6de9491360c993a8ea1b89a726e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/189820
Reviewed-by: Caolán McNamara <caolan.mcnam...@collabora.com>
Tested-by: Caolán McNamara <caolan.mcnam...@collabora.com>
diff --git a/configmgr/source/valueparser.cxx b/configmgr/source/valueparser.cxx
index 0b07efda235b..677584e343b5 100644
--- a/configmgr/source/valueparser.cxx
+++ b/configmgr/source/valueparser.cxx
@@ -195,7 +195,6 @@ template< typename T > css::uno::Any parseListValue(
}
if (text.length != 0) {
for (xmlreader::Span t(text);;) {
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
sal_Int32 i = rtl_str_indexOfStr_WithLength(
t.begin, t.length, sep.begin, sep.length);
T val;
diff --git a/cpputools/source/unoexe/unoexe.cxx
b/cpputools/source/unoexe/unoexe.cxx
index c7791538d3b6..d4bedbf7f471 100644
--- a/cpputools/source/unoexe/unoexe.cxx
+++ b/cpputools/source/unoexe/unoexe.cxx
@@ -464,7 +464,6 @@ SAL_IMPLEMENT_MAIN()
xContext, aImplName, aLocation, aServiceName, aInitParams,
bSingleInstance, aInstanceName ) );
- // coverity[loop_top] - not really an infinite loop, we can be
instructed to exit via the connection
for (;;)
{
// accepting
diff --git a/dbaccess/source/filter/hsqldb/utils.cxx
b/dbaccess/source/filter/hsqldb/utils.cxx
index ac479d9e05a2..2058bfdeaf57 100644
--- a/dbaccess/source/filter/hsqldb/utils.cxx
+++ b/dbaccess/source/filter/hsqldb/utils.cxx
@@ -82,7 +82,7 @@ OUString utils::getTableNameFromStmt(std::u16string_view sSql)
if (wordIter->indexOf("\"") >= 0)
{
size_t nAposBegin = sSql.find('"');
- assert(nAposBegin != std::u16string_view::npos); // make coverity happy
+ assert(nAposBegin != std::u16string_view::npos);
size_t nAposEnd = nAposBegin;
bool bProperEndAposFound = false;
while (!bProperEndAposFound)
diff --git a/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
b/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
index 3faf1bb9593a..cb3759f275a6 100644
--- a/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
+++ b/desktop/source/deployment/gui/dp_gui_extlistbox.cxx
@@ -433,7 +433,6 @@ void ExtensionBox_Impl::DrawRow(vcl::RenderContext&
rRenderContext, const tools:
auto aTextHeight = rRenderContext.GetTextHeight();
// Get max title width
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
auto nMaxTitleWidth = rRect.GetWidth() - ICON_OFFSET;
nMaxTitleWidth -= (2 * SMALL_ICON_SIZE) + (4 * SPACE_BETWEEN);
rRenderContext.SetFont(aStdFont);
diff --git
a/desktop/source/deployment/registry/configuration/dp_configuration.cxx
b/desktop/source/deployment/registry/configuration/dp_configuration.cxx
index 67acb9c7cdc7..e3789008306f 100644
--- a/desktop/source/deployment/registry/configuration/dp_configuration.cxx
+++ b/desktop/source/deployment/registry/configuration/dp_configuration.cxx
@@ -581,7 +581,6 @@ OUString replaceOrigin(
size_t write_pos = 0;
while (nBytes > 0)
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
sal_Int32 index = rtl_str_indexOfChar_WithLength( pBytes, nBytes, '%'
);
if (index < 0) {
if (! use_filtered) // opt
diff --git a/desktop/unx/source/start.c b/desktop/unx/source/start.c
index c80980389181..fce975227fa5 100644
--- a/desktop/unx/source/start.c
+++ b/desktop/unx/source/start.c
@@ -516,7 +516,6 @@ static ProgressStatus read_percent(ChildInfo const *info,
int *pPercent)
memmove(pBuffer, pNext, nNotProcessed);
/* read data */
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
ssize_t nThisRead = read(child_info_get_status_fd(info),
pBuffer + nNotProcessed, BUFFER_LEN -
nNotProcessed);
diff --git a/drawinglayer/source/primitive2d/textbreakuphelper.cxx
b/drawinglayer/source/primitive2d/textbreakuphelper.cxx
index 7356d606cb92..eac53589a7ff 100644
--- a/drawinglayer/source/primitive2d/textbreakuphelper.cxx
+++ b/drawinglayer/source/primitive2d/textbreakuphelper.cxx
@@ -120,7 +120,6 @@ namespace drawinglayer::primitive2d
}
// add text transformation to new transformation
- // coverity[swapped_arguments : FALSE] - this is in the correct
order
aNewTransform *= maDecTrans.getB2DHomMatrix();
// callback to allow evtl. changes
diff --git a/editeng/source/editeng/impedit.cxx
b/editeng/source/editeng/impedit.cxx
index e93b0c565322..d29c8af18d59 100644
--- a/editeng/source/editeng/impedit.cxx
+++ b/editeng/source/editeng/impedit.cxx
@@ -150,7 +150,6 @@ tools::Rectangle LOKSpecialPositioning::GetWindowPos(const
tools::Rectangle& rDo
else
{
Point aNewPos(aPos.X() - aSz.Height(), aPos.Y());
- // coverity[swapped_arguments : FALSE] - this is in the correct order
aRect = tools::Rectangle(aNewPos, Size(aSz.Height(), aSz.Width()));
}
return aRect;
@@ -820,7 +819,6 @@ tools::Rectangle ImpEditView::GetWindowPos( const
tools::Rectangle& rDocRect ) c
else
{
Point aNewPos( aPos.X()-aSz.Height(), aPos.Y() );
- // coverity[swapped_arguments : FALSE] - this is in the correct order
aRect = tools::Rectangle( aNewPos, Size( aSz.Height(), aSz.Width() ) );
}
return aRect;
diff --git a/editeng/source/editeng/impedit.hxx
b/editeng/source/editeng/impedit.hxx
index ba109b311ace..4fb322ddea46 100644
--- a/editeng/source/editeng/impedit.hxx
+++ b/editeng/source/editeng/impedit.hxx
@@ -221,9 +221,7 @@ public:
tools::Long GetVisDocLeft() const { return maVisDocStartPos.X(); }
tools::Long GetVisDocTop() const { return maVisDocStartPos.Y(); }
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetVisDocRight() const { return maVisDocStartPos.X() +
(!IsVertical() ? maOutArea.GetWidth() : maOutArea.GetHeight()); }
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetVisDocBottom() const { return maVisDocStartPos.Y() +
(!IsVertical() ? maOutArea.GetHeight() : maOutArea.GetWidth()); }
tools::Rectangle GetVisDocArea() const;
@@ -389,9 +387,7 @@ public:
tools::Long GetVisDocLeft() const { return maVisDocStartPos.X(); }
tools::Long GetVisDocTop() const { return maVisDocStartPos.Y(); }
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetVisDocRight() const { return maVisDocStartPos.X() + (
!IsVertical() ? maOutputArea.GetWidth() : maOutputArea.GetHeight() ); }
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetVisDocBottom() const { return maVisDocStartPos.Y() + (
!IsVertical() ? maOutputArea.GetHeight() : maOutputArea.GetWidth() ); }
tools::Rectangle GetVisDocArea() const;
diff --git a/embeddedobj/source/msole/ownview.cxx
b/embeddedobj/source/msole/ownview.cxx
index b9443998e4aa..480f0f67dbf5 100644
--- a/embeddedobj/source/msole/ownview.cxx
+++ b/embeddedobj/source/msole/ownview.cxx
@@ -409,7 +409,6 @@ void OwnView_Impl::CreateNative()
{
sal_uInt8 const aClassID[] =
{ 0x00, 0x03, 0x00, 0x0C, 0x00, 0x00, 0x00, 0x00, 0xC0, 0x00,
0x00, 0x00, 0x00, 0x00, 0x00, 0x46 };
- // coverity[overrun-buffer-arg : FALSE] - coverity has difficulty
with css::uno::Sequence
uno::Sequence< sal_Int8 > aPackageClassID(
reinterpret_cast<sal_Int8 const *>(aClassID), 16 );
uno::Reference< io::XStream > xSubStream;
diff --git a/extensions/source/update/check/updatecheck.cxx
b/extensions/source/update/check/updatecheck.cxx
index 97aa043d3f87..b22ff4c0a843 100644
--- a/extensions/source/update/check/updatecheck.cxx
+++ b/extensions/source/update/check/updatecheck.cxx
@@ -747,7 +747,6 @@ UpdateCheck::initialize(const uno::Sequence<
beans::NamedValue >& rValues,
bool downloadPaused = aModel.isDownloadPaused();
enableDownload(true, downloadPaused);
- // coverity[lock_order : FALSE] - incorrect report of lock
order error with std::recursive_mutex
setUIState(downloadPaused ? UPDATESTATE_DOWNLOAD_PAUSED :
UPDATESTATE_DOWNLOADING);
}
@@ -780,7 +779,6 @@ UpdateCheck::initialize(const uno::Sequence<
beans::NamedValue >& rValues,
setUIState( UPDATESTATE_DOWNLOAD_AVAIL );
else
{
- // coverity[lock_order : FALSE] - incorrect report of lock
order error with std::recursive_mutex
setUIState(getUIState(m_aUpdateInfo));
}
}
diff --git a/filter/source/msfilter/mscodec.cxx
b/filter/source/msfilter/mscodec.cxx
index a75718b4d49b..afd5b50dc4e2 100644
--- a/filter/source/msfilter/mscodec.cxx
+++ b/filter/source/msfilter/mscodec.cxx
@@ -186,7 +186,6 @@ bool MSCodec_Xor95::InitCodec( const uno::Sequence<
beans::NamedValue >& aData )
uno::Sequence< beans::NamedValue > MSCodec_Xor95::GetEncryptionData()
{
::comphelper::SequenceAsHashMap aHashData;
- // coverity[overrun-buffer-arg : FALSE] - coverity has difficulty with
css::uno::Sequence
aHashData[ u"XOR95EncryptionKey"_ustr ] <<= uno::Sequence<sal_Int8>(
reinterpret_cast<sal_Int8*>(mpnKey), 16 );
aHashData[ u"XOR95BaseKey"_ustr ] <<= static_cast<sal_Int16>(mnKey);
aHashData[ u"XOR95PasswordHash"_ustr ] <<= static_cast<sal_Int16>(mnHash);
diff --git a/filter/source/svg/svgexport.cxx b/filter/source/svg/svgexport.cxx
index 1ddc6b47b8b1..10b5f65ba8c7 100644
--- a/filter/source/svg/svgexport.cxx
+++ b/filter/source/svg/svgexport.cxx
@@ -2789,7 +2789,6 @@ IMPL_LINK( SVGFilter, CalcFieldHdl, EditFieldInfo*,
pInfo, void )
sDate.append( SvxDateField::GetFormatted(
aDate, eDateFormat, *pNumberFormatter, eLang ) );
}
break;
- // coverity[dead_error_begin] - following
conditions exist to avoid compiler warning
case SvxDateFormat::StdSmall:
case SvxDateFormat::A: // 13.02.96
case SvxDateFormat::B: // 13.02.1996
diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx
index 3b1a7019cb13..81cf70644548 100644
--- a/filter/source/svg/svgwriter.cxx
+++ b/filter/source/svg/svgwriter.cxx
@@ -2643,7 +2643,6 @@ void SVGActionWriter::ImplWriteText( const Point& rPos,
const OUString& rText,
if ( aTextColor == COL_BLACK )
aTextColor = COL_WHITE;
- // coverity[copy_paste_error : FALSE] - aReliefColor depending on
aTextColor is correct
if (aTextColor == COL_WHITE)
aReliefColor = COL_BLACK;
diff --git a/formula/source/core/api/FormulaCompiler.cxx
b/formula/source/core/api/FormulaCompiler.cxx
index 2ced8cfabe5f..9e9e90cd67c3 100644
--- a/formula/source/core/api/FormulaCompiler.cxx
+++ b/formula/source/core/api/FormulaCompiler.cxx
@@ -2166,7 +2166,6 @@ void FormulaCompiler::IntersectionLine()
FormulaTokenRef pIntersect( new FormulaByteToken(
ocIntersect));
// Replace ocSpaces with ocIntersect so that when switching
// formula syntax the correct operator string is created.
- // coverity[freed_arg : FALSE] - FormulaTokenRef has a ref so
ReplaceToken won't delete pIntersect
pArr->ReplaceToken( nCodeIndex, pIntersect.get(),
FormulaTokenArray::ReplaceMode::CODE_ONLY);
PutCode( pIntersect);
}
diff --git a/fpicker/source/office/OfficeFilePicker.cxx
b/fpicker/source/office/OfficeFilePicker.cxx
index 72c2c98af7a9..b27e8c30d481 100644
--- a/fpicker/source/office/OfficeFilePicker.cxx
+++ b/fpicker/source/office/OfficeFilePicker.cxx
@@ -422,7 +422,6 @@ sal_Int16 SvtFilePicker::implExecutePicker( )
// now we are ready to execute the dialog
sal_Int16 nRet = m_xDlg->run();
- // coverity[check_after_deref] - the execution of the dialog yields, so it
is possible the at this point the window or the dialog is closed
if (m_xDlg)
m_xDlg->SetFileCallback( nullptr );
diff --git a/framework/source/accelerators/acceleratorconfiguration.cxx
b/framework/source/accelerators/acceleratorconfiguration.cxx
index 1b3fc6cec9b2..43a62bfb21e6 100644
--- a/framework/source/accelerators/acceleratorconfiguration.cxx
+++ b/framework/source/accelerators/acceleratorconfiguration.cxx
@@ -1076,7 +1076,6 @@ void XCUBasedAcceleratorConfiguration::impl_ts_save(bool
bPreferred)
// take over all changes into the original container
SolarMutexGuard g;
- // coverity[check_after_deref] - confusing but correct
if (m_pPrimaryWriteCache)
{
m_aPrimaryReadCache = *m_pPrimaryWriteCache;
@@ -1112,7 +1111,6 @@ void XCUBasedAcceleratorConfiguration::impl_ts_save(bool
bPreferred)
// take over all changes into the original container
SolarMutexGuard g;
- // coverity[check_after_deref] - confusing but correct
if (m_pSecondaryWriteCache)
{
m_aSecondaryReadCache = *m_pSecondaryWriteCache;
diff --git a/oox/source/export/ColorPropertySet.cxx
b/oox/source/export/ColorPropertySet.cxx
index 714870d93e36..5d3b44efbe21 100644
--- a/oox/source/export/ColorPropertySet.cxx
+++ b/oox/source/export/ColorPropertySet.cxx
@@ -155,7 +155,6 @@ PropertyState SAL_CALL ColorPropertySet::getPropertyState(
const OUString& /* Pr
Sequence< PropertyState > SAL_CALL ColorPropertySet::getPropertyStates( const
Sequence< OUString >& /* aPropertyName */ )
{
PropertyState aState = PropertyState_DIRECT_VALUE;
- // coverity[overrun-buffer-arg : FALSE] - coverity has difficulty with
css::uno::Sequence
return Sequence<PropertyState>(&aState, 1);
}
diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx
index 168dda3e9704..a50226db8415 100644
--- a/sc/source/core/data/column3.cxx
+++ b/sc/source/core/data/column3.cxx
@@ -580,7 +580,6 @@ void ScColumn::AttachFormulaCells(
sc::StartListeningContext& rCxt, SCROW nRow1,
{
nRow1 = std::min( nRow1, rCell.GetSharedTopRow());
SCROW nCellLen = rCell.GetSharedLength();
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
const auto nEndRow = rCell.GetSharedTopRow() + nCellLen;
if (nRow2 < nEndRow)
{
diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index e28993653fee..d24f950a6394 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1772,7 +1772,6 @@ void ScTabView::DoHSplit(tools::Long nSplitPos)
}
else
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
SCCOL nOldDelta = aViewData.GetPosX( SC_SPLIT_LEFT );
tools::Long nLeftWidth = nSplitPos -
pRowBar[SC_SPLIT_BOTTOM]->GetSizePixel().Width();
if ( nLeftWidth < 0 ) nLeftWidth = 0;
@@ -1839,10 +1838,7 @@ void ScTabView::DoVSplit(tools::Long nSplitPos)
else
{
if ( aOldMode == SC_SPLIT_NONE )
- {
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
nOldDelta = aViewData.GetPosY( SC_SPLIT_BOTTOM );
- }
else
nOldDelta = aViewData.GetPosY( SC_SPLIT_TOP );
diff --git a/scaddins/source/analysis/analysis.cxx
b/scaddins/source/analysis/analysis.cxx
index 5361bc7865d1..80166157baa3 100644
--- a/scaddins/source/analysis/analysis.cxx
+++ b/scaddins/source/analysis/analysis.cxx
@@ -407,7 +407,6 @@ sal_Int32 SAL_CALL AnalysisAddIn::getWeeknum( const
uno::Reference< beans::XProp
DaysToDate( nDate, nDay, nMonth, nYear );
sal_Int32 nFirstInYear = DateToDays( 1, 1, nYear );
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
sal_uInt16 nFirstDayInYear = GetDayOfWeek( nFirstInYear );
return ( nDate - nFirstInYear + ( ( nMode == 1 )? ( nFirstDayInYear + 1 )
% 7 : nFirstDayInYear ) ) / 7 + 1;
diff --git a/sd/source/ui/remotecontrol/BufferedStreamSocket.cxx
b/sd/source/ui/remotecontrol/BufferedStreamSocket.cxx
index 2834bb74a9f9..e2fe96fc383e 100644
--- a/sd/source/ui/remotecontrol/BufferedStreamSocket.cxx
+++ b/sd/source/ui/remotecontrol/BufferedStreamSocket.cxx
@@ -107,10 +107,7 @@ sal_Int32 BufferedStreamSocket::readLine( OString& aLine )
sal_Int32 nRet;
if (!usingCSocket)
- {
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
nRet = StreamSocket::recv( &aBuffer[aRead], 100 );
- }
else
nRet = ::recv( mSocket, &aBuffer[aRead], 100, 0 );
diff --git a/svtools/source/brwbox/datwin.cxx b/svtools/source/brwbox/datwin.cxx
index 6e608ef63683..35d52cb2a187 100644
--- a/svtools/source/brwbox/datwin.cxx
+++ b/svtools/source/brwbox/datwin.cxx
@@ -55,7 +55,6 @@ void ButtonFrame::Draw( OutputDevice& rDev )
if ( !aText.isEmpty() )
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
OUString aVal = rDev.GetEllipsisString(aText,aInnerRect.GetWidth() -
2*MIN_COLUMNWIDTH);
vcl::Font aFont( rDev.GetFont() );
diff --git a/svx/source/stbctrls/zoomsliderctrl.cxx
b/svx/source/stbctrls/zoomsliderctrl.cxx
index 5bae895870e0..5acfce63a2f6 100644
--- a/svx/source/stbctrls/zoomsliderctrl.cxx
+++ b/svx/source/stbctrls/zoomsliderctrl.cxx
@@ -122,7 +122,6 @@ sal_uInt16 SvxZoomSliderControl::Offset2Zoom( tools::Long
nOffset ) const
// returns the offset to the left control border
tools::Long SvxZoomSliderControl::Zoom2Offset( sal_uInt16 nCurrentZoom ) const
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
const tools::Long nControlWidth = getControlRect().GetWidth();
tools::Long nRet = nSliderXOffset;
diff --git a/sw/inc/view.hxx b/sw/inc/view.hxx
index 1ebd27a390bf..7380d225139c 100644
--- a/sw/inc/view.hxx
+++ b/sw/inc/view.hxx
@@ -294,9 +294,7 @@ class SW_DLLPUBLIC SwView: public SfxViewShell
DECL_DLLPRIVATE_LINK( TimeoutHdl, Timer*, void );
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetXScroll() const { return (m_aVisArea.GetWidth() * nScrollX)
/ 100; }
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long GetYScroll() const { return (m_aVisArea.GetHeight() *
nScrollY) / 100; }
SAL_DLLPRIVATE Point AlignToPixel(const Point& rPt) const;
diff --git a/sw/source/core/view/pagepreviewlayout.cxx
b/sw/source/core/view/pagepreviewlayout.cxx
index 1b0445f7489f..513120033f96 100644
--- a/sw/source/core/view/pagepreviewlayout.cxx
+++ b/sw/source/core/view/pagepreviewlayout.cxx
@@ -937,7 +937,6 @@ SwTwips SwPagePreviewLayout::GetWinPagesScrollAmount(
}
else
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
nScrollAmount = _nWinPagesToScroll *
maPaintedPreviewDocRect.GetHeight();
}
diff --git a/sw/source/filter/ww8/writerwordglue.cxx
b/sw/source/filter/ww8/writerwordglue.cxx
index 449cb60de498..0db302e5924b 100644
--- a/sw/source/filter/ww8/writerwordglue.cxx
+++ b/sw/source/filter/ww8/writerwordglue.cxx
@@ -746,7 +746,6 @@ namespace sw
if ( rDT.GetDate() == 0 )
return 0;
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
sal_uInt32 nDT = ( rDT.GetDayOfWeek() + 1 ) % 7;
nDT <<= 9;
nDT += ( rDT.GetYear() - 1900 ) & 0x1ff;
diff --git a/tools/source/datetime/tdate.cxx b/tools/source/datetime/tdate.cxx
index 3548f08f06f8..92da1cd3d42a 100644
--- a/tools/source/datetime/tdate.cxx
+++ b/tools/source/datetime/tdate.cxx
@@ -209,7 +209,6 @@ sal_uInt16 Date::GetDayOfYear() const
sal_uInt16 Date::GetWeekOfYear( DayOfWeek eStartDay,
sal_Int16 nMinimumNumberOfDaysInWeek ) const
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
short n1WDay = static_cast<short>(Date( 1, 1, GetYear() ).GetDayOfWeek());
short nDayOfYear = static_cast<short>(GetDayOfYear());
diff --git a/vcl/source/app/svcache.cxx b/vcl/source/app/svcache.cxx
index 32c972c1e2ea..5c6d1e07cde6 100644
--- a/vcl/source/app/svcache.cxx
+++ b/vcl/source/app/svcache.cxx
@@ -98,7 +98,6 @@ CacheMemory::CacheMemory()
#if defined FREEBSD || defined NETBSD || defined OPENBSD || defined DRAGONFLY
|| defined HAIKU
mnPageSize = getpagesize();
#else
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
mnPageSize = sysconf(_SC_PAGESIZE);
#endif
#elif defined _WIN32
diff --git a/vcl/source/outdev/map.cxx b/vcl/source/outdev/map.cxx
index 5ef6e922e6aa..da9b3c4e6878 100644
--- a/vcl/source/outdev/map.cxx
+++ b/vcl/source/outdev/map.cxx
@@ -277,7 +277,6 @@ tools::Long OutputDevice::ImplLogicXToDevicePixel(
tools::Long nX ) const
if ( !mbMap )
return nX+mnOutOffX;
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
return ImplLogicToPixel( nX + maMapRes.mnMapOfsX, mnDPIX,
maMapRes.mnMapScNumX, maMapRes.mnMapScDenomX
)+mnOutOffX+mnOutOffOrigX;
}
@@ -287,7 +286,6 @@ tools::Long OutputDevice::ImplLogicYToDevicePixel(
tools::Long nY ) const
if ( !mbMap )
return nY+mnOutOffY;
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
return ImplLogicToPixel( nY + maMapRes.mnMapOfsY, mnDPIY,
maMapRes.mnMapScNumY, maMapRes.mnMapScDenomY
)+mnOutOffY+mnOutOffOrigY;
}
diff --git a/vcl/source/window/menu.cxx b/vcl/source/window/menu.cxx
index 76e4eeae10eb..1fbb5b2f3179 100644
--- a/vcl/source/window/menu.cxx
+++ b/vcl/source/window/menu.cxx
@@ -2268,7 +2268,6 @@ tools::Long Menu::GetIndexForPoint( const Point& rPoint,
sal_uInt16& rItemID ) c
ImplFillLayoutData();
if( mpLayoutData )
{
- // coverity[ tainted_data_return : FALSE ] version 2023.12.2
tools::Long nIndex = mpLayoutData->GetIndexForPoint( rPoint );
for( size_t i = 0; i < mpLayoutData->m_aLineIndices.size(); i++ )
{
diff --git a/writerperfect/source/common/WPXSvInputStream.cxx
b/writerperfect/source/common/WPXSvInputStream.cxx
index c4fa9c56614a..03c5ae2961ac 100644
--- a/writerperfect/source/common/WPXSvInputStream.cxx
+++ b/writerperfect/source/common/WPXSvInputStream.cxx
@@ -739,7 +739,6 @@ WPXSvInputStream::~WPXSvInputStream() {}
long WPXSvInputStream::tell()
{
- // coverity[tainted_data_return : FALSE] retVal is considered safe now
tools::Long retVal = tellImpl();
if (retVal < 0)
return -1;
