New Defects reported by Coverity Scan for LibreOffice
scan-admin at coverity.com
scan-admin at coverity.com
Mon Jul 1 20:55:25 UTC 2024
Hi,
Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.
300 new defect(s) introduced to LibreOffice found with Coverity Scan.
175 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 20 of 300 defect(s)
** CID 1608034: (LOCK)
/filter/source/config/cache/typedetection.cxx: 1046 in filter::config::TypeDetection::impl_detectTypeFlatAndDeep(utl::MediaDescriptor &, const std::vector<filter::config::FlatDetectionInfo, std::allocator<filter::config::FlatDetectionInfo>> &, bool, rtl::OUString &)()
/filter/source/config/cache/typedetection.cxx: 1045 in filter::config::TypeDetection::impl_detectTypeFlatAndDeep(utl::MediaDescriptor &, const std::vector<filter::config::FlatDetectionInfo, std::allocator<filter::config::FlatDetectionInfo>> &, bool, rtl::OUString &)()
________________________________________________________________________________________________________
*** CID 1608034: (LOCK)
/filter/source/config/cache/typedetection.cxx: 1046 in filter::config::TypeDetection::impl_detectTypeFlatAndDeep(utl::MediaDescriptor &, const std::vector<filter::config::FlatDetectionInfo, std::allocator<filter::config::FlatDetectionInfo>> &, bool, rtl::OUString &)()
1040
1041 OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
1042
1043 // d)
1044 if (!sDeepType.isEmpty())
1045 return sDeepType;
>>> CID 1608034: (LOCK)
>>> "~unique_lock" unlocks "aLock" while it is unlocked. [Note: The source code implementation of the function has been overridden by a builtin model.]
1046 }
1047 catch(const css::container::NoSuchElementException&)
1048 {}
1049 // e)
1050 }
1051
/filter/source/config/cache/typedetection.cxx: 1045 in filter::config::TypeDetection::impl_detectTypeFlatAndDeep(utl::MediaDescriptor &, const std::vector<filter::config::FlatDetectionInfo, std::allocator<filter::config::FlatDetectionInfo>> &, bool, rtl::OUString &)()
1039 }
1040
1041 OUString sDeepType = impl_askDetectService(sDetectService, rDescriptor);
1042
1043 // d)
1044 if (!sDeepType.isEmpty())
>>> CID 1608034: (LOCK)
>>> "~unique_lock" unlocks "aLock" while it is unlocked. [Note: The source code implementation of the function has been overridden by a builtin model.]
1045 return sDeepType;
1046 }
1047 catch(const css::container::NoSuchElementException&)
1048 {}
1049 // e)
1050 }
** CID 1608033: Memory - illegal accesses (INTEGER_OVERFLOW)
/sc/source/core/data/table3.cxx: 2056 in ScTable::DoSubTotals(ScSubTotalParam &)()
________________________________________________________________________________________________________
*** CID 1608033: Memory - illegal accesses (INTEGER_OVERFLOW)
/sc/source/core/data/table3.cxx: 2056 in ScTable::DoSubTotals(ScSubTotalParam &)()
2050
2051 for (sal_uInt16 nLevel=0; nLevel<nLevelCount && bSpaceLeft; nLevel++)
2052 {
2053 aRowEntry.nGroupNo = nLevelCount - nLevel - 1;
2054
2055 // how many results per level
>>> CID 1608033: Memory - illegal accesses (INTEGER_OVERFLOW)
>>> "aRowEntry.nGroupNo", which might have underflowed, is passed to "rParam.nSubTotals[aRowEntry.nGroupNo]".
2056 SCCOL nResCount = rParam.nSubTotals[aRowEntry.nGroupNo];
2057 // result functions
2058 ScSubTotalFunc* pResFunc = rParam.pFunctions[aRowEntry.nGroupNo].get();
2059
2060 if (nResCount > 0) // otherwise only sort
2061 {
** CID 1608032: (INTEGER_OVERFLOW)
/sw/source/core/doc/gctable.cxx: 446 in lcl_MergeGCLine(SwTableLine *, <unnamed>::GCLinePara *)()
/sw/source/core/doc/gctable.cxx: 444 in lcl_MergeGCLine(SwTableLine *, <unnamed>::GCLinePara *)()
________________________________________________________________________________________________________
*** CID 1608032: (INTEGER_OVERFLOW)
/sw/source/core/doc/gctable.cxx: 446 in lcl_MergeGCLine(SwTableLine *, <unnamed>::GCLinePara *)()
440 nBoxes = pLn->GetTabBoxes().size();
441 }
442
443 // ATTENTION: The number of boxes can change!
444 for( SwTableBoxes::size_type nLen = 0; nLen < pLn->GetTabBoxes().size(); ++nLen )
445 if( !lcl_MergeGCBox( pLn->GetTabBoxes()[nLen], pGCPara ))
>>> CID 1608032: (INTEGER_OVERFLOW)
>>> Expression "--nLen", which is equal to 18446744073709551615, where "nLen" is known to be equal to 0, underflows the type that receives it, an unsigned integer 64 bits wide.
446 --nLen;
447 }
448 return true;
449 }
450
451 // Clean structure a bit
/sw/source/core/doc/gctable.cxx: 444 in lcl_MergeGCLine(SwTableLine *, <unnamed>::GCLinePara *)()
438
439 pLn = pLine; // and set up anew
440 nBoxes = pLn->GetTabBoxes().size();
441 }
442
443 // ATTENTION: The number of boxes can change!
>>> CID 1608032: (INTEGER_OVERFLOW)
>>> Expression "++nLen", which is equal to 0, where "nLen" is known to be equal to 18446744073709551615, overflows the type that receives it, an unsigned integer 64 bits wide.
444 for( SwTableBoxes::size_type nLen = 0; nLen < pLn->GetTabBoxes().size(); ++nLen )
445 if( !lcl_MergeGCBox( pLn->GetTabBoxes()[nLen], pGCPara ))
446 --nLen;
447 }
448 return true;
449 }
** CID 1608031: SpotBugs: Bad practice (FB.CT_CONSTRUCTOR_THROW)
/reportbuilder/java/org/libreoffice/report/pentaho/parser/rpt/FormatConditionReadHandler.java: 46 in org.libreoffice.report.pentaho.parser.rpt.FormatConditionReadHandler.<init>(org.libreoffice.report.pentaho.model.ReportElement)()
________________________________________________________________________________________________________
*** CID 1608031: SpotBugs: Bad practice (FB.CT_CONSTRUCTOR_THROW)
/reportbuilder/java/org/libreoffice/report/pentaho/parser/rpt/FormatConditionReadHandler.java: 46 in org.libreoffice.report.pentaho.parser.rpt.FormatConditionReadHandler.<init>(org.libreoffice.report.pentaho.model.ReportElement)()
40 private final ReportElement element;
41
42 public FormatConditionReadHandler(final ReportElement element)
43 {
44 if (element == null)
45 {
>>> CID 1608031: SpotBugs: Bad practice (FB.CT_CONSTRUCTOR_THROW)
>>> Exception thrown in class org.libreoffice.report.pentaho.parser.rpt.FormatConditionReadHandler at new org.libreoffice.report.pentaho.parser.rpt.FormatConditionReadHandler(ReportElement) will leave the constructor. The object under construction remains partially initialized and may be vulnerable to Finalizer attacks.
46 throw new NullPointerException();
47 }
48 this.element = element;
49 }
50
51 @Override
** CID 1608030: Data race undermines locking (LOCK_EVASION)
/sw/source/core/unocore/unochart.cxx: 2665 in SwChartLabeledDataSequence::dispose()()
________________________________________________________________________________________________________
*** CID 1608030: Data race undermines locking (LOCK_EVASION)
/sw/source/core/unocore/unochart.cxx: 2665 in SwChartLabeledDataSequence::dispose()()
2659 bMustDispose = !m_bDisposed;
2660 if (!m_bDisposed)
2661 m_bDisposed = true;
2662 }
2663 if (bMustDispose)
2664 {
>>> CID 1608030: Data race undermines locking (LOCK_EVASION)
>>> Thread1 sets "m_bDisposed" to a new value. Now the two threads have an inconsistent view of "m_bDisposed" and updates to fields correlated with "m_bDisposed" may be lost.
2665 m_bDisposed = true;
2666
2667 // require listeners to release references to this object
2668 lang::EventObject aEvtObj( static_cast< chart2::data::XLabeledDataSequence * >(this) );
2669 std::unique_lock aGuard( GetChartMutex() );
2670 m_aModifyListeners.disposeAndClear( aGuard, aEvtObj );
** CID 1608029: Uninitialized variables (UNINIT)
________________________________________________________________________________________________________
*** CID 1608029: Uninitialized variables (UNINIT)
/comphelper/source/property/propshlp.cxx: 806 in comphelper::OPropertySetHelper::removePropertiesChangeListener(const com::sun::star::uno::Reference<com::sun::star::beans::XPropertiesChangeListener> &)()
800
801 // XMultiPropertySet
802 void OPropertySetHelper::removePropertiesChangeListener(
803 const Reference<XPropertiesChangeListener>& rListener)
804 {
805 std::unique_lock g(m_aMutex);
>>> CID 1608029: Uninitialized variables (UNINIT)
>>> Using uninitialized value "g._M_owns" when calling "removeInterface".
806 maPropertiesChangeListeners.removeInterface(g, rListener);
807 }
808
809 // XMultiPropertySet
810 void OPropertySetHelper::firePropertiesChangeEvent(
811 const Sequence<OUString>& rPropertyNames, const Reference<XPropertiesChangeListener>& rListener)
** CID 1608028: Concurrent data access violations (MISSING_LOCK)
/ucbhelper/source/provider/resultset.cxx: 1166 in ucbhelper::ResultSet::queryContent()()
________________________________________________________________________________________________________
*** CID 1608028: Concurrent data access violations (MISSING_LOCK)
/ucbhelper/source/provider/resultset.cxx: 1166 in ucbhelper::ResultSet::queryContent()()
1160
1161
1162 // virtual
1163 uno::Reference< css::ucb::XContent > SAL_CALL
1164 ResultSet::queryContent()
1165 {
>>> CID 1608028: Concurrent data access violations (MISSING_LOCK)
>>> Accessing "this->m_pImpl->m_bAfterLast" without holding lock "ucbhelper::ResultSet_Impl.m_aMutex". Elsewhere, "ucbhelper::ResultSet_Impl.m_bAfterLast" is written to with "ResultSet_Impl.m_aMutex" held 14 out of 14 times (12 of these accesses strongly imply that it is necessary).
1166 if ( m_pImpl->m_nPos && !m_pImpl->m_bAfterLast )
1167 return m_pImpl->m_xDataSupplier->queryContent( m_pImpl->m_nPos - 1 );
1168
1169 return uno::Reference< css::ucb::XContent >();
1170 }
1171
** CID 1608027: Uninitialized variables (UNINIT)
________________________________________________________________________________________________________
*** CID 1608027: Uninitialized variables (UNINIT)
/chart2/source/model/main/ChartModel_Persistence.cxx: 780 in chart::ChartModel::removeStorageChangeListener(const com::sun::star::uno::Reference<com::sun::star::document::XStorageChangeListener> &)()
774 void SAL_CALL ChartModel::removeStorageChangeListener( const Reference< document::XStorageChangeListener >& xListener )
775 {
776 if( m_aLifeTimeManager.impl_isDisposedOrClosed(false) )
777 return; //behave passive if already disposed or closed
778
779 std::unique_lock aGuard(m_aLifeTimeManager.m_aAccessMutex);
>>> CID 1608027: Uninitialized variables (UNINIT)
>>> Using uninitialized value "aGuard._M_owns" when calling "removeInterface".
780 m_aLifeTimeManager.m_aStorageChangeListeners.removeInterface(aGuard, xListener );
781 }
782
783 } // namespace chart
784
** CID 1608026: SpotBugs: Bad practice (FB.PA_PUBLIC_PRIMITIVE_ATTRIBUTE)
/qadevOOo/tests/java/ifc/container/_XChild.java: 40 in ()
________________________________________________________________________________________________________
*** CID 1608026: SpotBugs: Bad practice (FB.PA_PUBLIC_PRIMITIVE_ATTRIBUTE)
/qadevOOo/tests/java/ifc/container/_XChild.java: 40 in ()
34 * </ul>
35 * @see com.sun.star.container.XChild
36 */
37 public class _XChild extends MultiMethodTest {
38
39 public XChild oObj = null;
>>> CID 1608026: SpotBugs: Bad practice (FB.PA_PUBLIC_PRIMITIVE_ATTRIBUTE)
>>> Primitive field ifc.container._XChild.gotten is public and set from inside the class, which makes it too exposed. Consider making it private to limit external accessibility.
40 public Object gotten = null;
41
42
43 /**
44 * Test calls the method and checks return value and that
45 * no exceptions were thrown. Parent returned is stored.<p>
** CID 1608025: Memory - illegal accesses (UNINIT)
________________________________________________________________________________________________________
*** CID 1608025: Memory - illegal accesses (UNINIT)
/slideshow/source/engine/eventqueue.cxx: 199 in slideshow::internal::EventQueue::process_(bool)()
193 // places, which should fire only once: after the
194 // initial fire() call, those events become inactive
195 // and return false on isCharged. This frees us from
196 // the need to prune queues of those inactive shells.
197 if( event.pEvent->isCharged() )
198 {
>>> CID 1608025: Memory - illegal accesses (UNINIT)
>>> Using uninitialized value "aGuard._M_owns" when calling "unlock".
199 aGuard.unlock();
200 try
201 {
202 SAL_INFO("slideshow.eventqueue", "firing event \""
203 << event.pEvent->GetDescription()
204 << "\" [" << event.pEvent.get()
** CID 1608024: Uninitialized variables (UNINIT)
________________________________________________________________________________________________________
*** CID 1608024: Uninitialized variables (UNINIT)
/sw/source/core/unocore/unochart.cxx: 2184 in SwChartDataSequence::addModifyListener(const com::sun::star::uno::Reference<com::sun::star::util::XModifyListener> &)()
2178
2179 void SAL_CALL SwChartDataSequence::addModifyListener(
2180 const uno::Reference< util::XModifyListener >& rxListener )
2181 {
2182 std::unique_lock aGuard( GetChartMutex() );
2183 if (!m_bDisposed && rxListener.is())
>>> CID 1608024: Uninitialized variables (UNINIT)
>>> Using uninitialized value "aGuard._M_owns" when calling "addInterface".
2184 m_aModifyListeners.addInterface( aGuard, rxListener );
2185 }
2186
2187 void SAL_CALL SwChartDataSequence::removeModifyListener(
2188 const uno::Reference< util::XModifyListener >& rxListener )
2189 {
** CID 1608023: SpotBugs: Bad practice (FB.PA_PUBLIC_MUTABLE_OBJECT_ATTRIBUTE)
/qadevOOo/tests/java/ifc/ucb/_XRemoteContentProviderAcceptor.java: 38 in ()
________________________________________________________________________________________________________
*** CID 1608023: SpotBugs: Bad practice (FB.PA_PUBLIC_MUTABLE_OBJECT_ATTRIBUTE)
/qadevOOo/tests/java/ifc/ucb/_XRemoteContentProviderAcceptor.java: 38 in ()
32 * </ul> <p>
33 * Test is <b> NOT </b> multithread compliant. <p>
34 * @see com.sun.star.ucb.XRemoteContentProviderAcceptor
35 */
36 public class _XRemoteContentProviderAcceptor extends MultiMethodTest {
37
>>> CID 1608023: SpotBugs: Bad practice (FB.PA_PUBLIC_MUTABLE_OBJECT_ATTRIBUTE)
>>> Mutable object-type field ifc.ucb._XRemoteContentProviderAcceptor.oObj is public, which makes it too exposed. Consider making it private to limit external accessibility.
38 public XRemoteContentProviderAcceptor oObj = null;
39
40 /**
41 * The simple <code>XRemoteContentProviderDoneListener</code>
42 * implementation.
43 */
** CID 1608022: Uninitialized members (UNINIT_CTOR)
/sc/source/ui/docshell/docsh4.cxx: 1979 in ScDocShell::ExecutePageStyle(const SfxViewShell &, SfxRequest &, short)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()
________________________________________________________________________________________________________
*** CID 1608022: Uninitialized members (UNINIT_CTOR)
/sc/source/ui/docshell/docsh4.cxx: 1979 in ScDocShell::ExecutePageStyle(const SfxViewShell &, SfxRequest &, short)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()
1973 ScAbstractDialogFactory* pFact = ScAbstractDialogFactory::Create();
1974
1975 VclPtr<SfxAbstractTabDialog> pDlg(pFact->CreateScStyleDlg(GetActiveDialogParent(), *pStyleSheet, true));
1976
1977 auto xRequest = std::make_shared<SfxRequest>(rReq);
1978 rReq.Ignore(); // the 'old' request is not relevant any more
>>> CID 1608022: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "this" is not initialized in this constructor nor in any functions that it calls.
1979 pDlg->StartExecuteAsync([this, pDlg, xRequest=std::move(xRequest), pStyleSheet, aOldData,
1980 aOldName, &rStyleSet, nCurTab, &rCaller, bUndo](sal_Int32 nResult) {
1981 if ( nResult == RET_OK )
1982 {
1983 const SfxItemSet* pOutSet = pDlg->GetOutputItemSet();
1984
** CID 1608021: (INTEGER_OVERFLOW)
/sw/source/core/undo/unattr.cxx: 678 in SwUndoResetAttr::RedoImpl(sw::UndoRedoContext &)()
/sw/source/core/undo/unattr.cxx: 679 in SwUndoResetAttr::RedoImpl(sw::UndoRedoContext &)()
________________________________________________________________________________________________________
*** CID 1608021: (INTEGER_OVERFLOW)
/sw/source/core/undo/unattr.cxx: 678 in SwUndoResetAttr::RedoImpl(sw::UndoRedoContext &)()
672 }
673 }
674 } else
675 nCnt = 0;
676 }
677 // found one, thus delete it
>>> CID 1608021: (INTEGER_OVERFLOW)
>>> Expression "nCnt--", which is equal to 65535, where "nCnt" is known to be equal to 0, underflows the type that receives it, an unsigned integer 16 bits wide.
678 if( nCnt-- ) {
679 rDoc.DeleteTOXMark( aArr[ nCnt ] );
680 }
681 }
682 }
683 break;
/sw/source/core/undo/unattr.cxx: 679 in SwUndoResetAttr::RedoImpl(sw::UndoRedoContext &)()
673 }
674 } else
675 nCnt = 0;
676 }
677 // found one, thus delete it
678 if( nCnt-- ) {
>>> CID 1608021: (INTEGER_OVERFLOW)
>>> "nCnt", which might have underflowed, is passed to "aArr[nCnt]". [Note: The source code implementation of the function has been overridden by a builtin model.]
679 rDoc.DeleteTOXMark( aArr[ nCnt ] );
680 }
681 }
682 }
683 break;
684 case RES_TXTATR_REFMARK:
** CID 1608020: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
/sc/source/core/tool/rangeseq.cxx: 232 in ScRangeToSequence::FillStringArray(com::sun::star::uno::Any &, const ScMatrix *, ScInterpreterContext &)()
________________________________________________________________________________________________________
*** CID 1608020: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
/sc/source/core/tool/rangeseq.cxx: 232 in ScRangeToSequence::FillStringArray(com::sun::star::uno::Any &, const ScMatrix *, ScInterpreterContext &)()
226 const Color* pColor;
227 rContext.NFGetOutputString( fVal, 0, aStr, &pColor );
228 }
229 pColAry[nCol] = aStr;
230 }
231
>>> CID 1608020: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
>>> "aColSeq" is copied in a call to copy assignment "operator =", when it could be moved instead.
232 pRowAry[nRow] = aColSeq;
233 }
234
235 rAny <<= aRowSeq;
236 return true;
237 }
** CID 1608019: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
/sc/source/ui/view/viewfunc.cxx: 738 in ScViewFunc::EnterData(short, int, short, const rtl::OUString &, const EditTextObject *, bool)()
________________________________________________________________________________________________________
*** CID 1608019: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
/sc/source/ui/view/viewfunc.cxx: 738 in ScViewFunc::EnterData(short, int, short, const rtl::OUString &, const EditTextObject *, bool)()
732 {
733 aCompPtr->SetExtendedErrorDetection( ScCompiler::EXTENDED_ERROR_DETECTION_NAME_BREAK );
734 }
735
736 OUString aFormula( rString );
737
>>> CID 1608019: Performance inefficiencies (COPY_INSTEAD_OF_MOVE)
>>> "rMark" is copied in call to copy constructor "ScMarkData", when it could be moved instead.
738 FormulaProcessingContext context_instance{
739 aPosPtr, aCompPtr, xModificator, nullptr, nullptr, pData,
740 rMark, *this, OUString(), aFormula, rString, nCol,
741 nRow, nTab, bMatrixExpand, bNumFmtChanged, bRecord
742 };
743
** CID 1608018: Uninitialized members (UNINIT_CTOR)
/sc/source/ui/view/tabvwshf.cxx: 1134 in ScTabViewShell::ExecuteAppendOrRenameTableDialog(const VclPtr<AbstractScStringInputDlg> &, const std::shared_ptr<SfxRequest> &, unsigned short)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()
________________________________________________________________________________________________________
*** CID 1608018: Uninitialized members (UNINIT_CTOR)
/sc/source/ui/view/tabvwshf.cxx: 1134 in ScTabViewShell::ExecuteAppendOrRenameTableDialog(const VclPtr<AbstractScStringInputDlg> &, const std::shared_ptr<SfxRequest> &, unsigned short)::[lambda(int) (instance 1)]::lambda(const [lambda(int) (instance 1)]&)()
1128
1129 void ScTabViewShell::ExecuteAppendOrRenameTableDialog(const VclPtr<AbstractScStringInputDlg>& pDlg,
1130 const std::shared_ptr<SfxRequest>& xReq,
1131 sal_uInt16 nSlot)
1132 {
1133 pDlg->StartExecuteAsync(
>>> CID 1608018: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "this" is not initialized in this constructor nor in any functions that it calls.
1134 [this, pDlg, xReq, nSlot] (sal_Int32 nResult)->void
1135 {
1136 if (DoAppendOrRenameTableDialog(nResult, pDlg, xReq, nSlot))
1137 ExecuteAppendOrRenameTableDialog(pDlg, xReq, nSlot);
1138 else
1139 pDlg->disposeOnce();
** CID 1608017: Uninitialized members (UNINIT_CTOR)
/sw/source/core/access/AccessibilityIssue.cxx: 313 in sw::AccessibilityIssue::quickFixIssue() const::[lambda(int) (instance 2)]::lambda([lambda(int) (instance 2)]&&)()
________________________________________________________________________________________________________
*** CID 1608017: Uninitialized members (UNINIT_CTOR)
/sw/source/core/access/AccessibilityIssue.cxx: 313 in sw::AccessibilityIssue::quickFixIssue() const::[lambda(int) (instance 2)]::lambda([lambda(int) (instance 2)]&&)()
307
308 SvxAbstractDialogFactory* pFact = SvxAbstractDialogFactory::Create();
309 VclPtr<AbstractSvxObjectTitleDescDialog> pDlg(pFact->CreateSvxObjectTitleDescDialog(
310 pWrtShell->GetView().GetFrameWeld(), aTitle, aDescription, isDecorative));
311
312 bResetAndQueue = false;
>>> CID 1608017: Uninitialized members (UNINIT_CTOR)
>>> Non-static class member "this" is not initialized in this constructor nor in any functions that it calls.
313 pDlg->StartExecuteAsync([this, pDlg, pObj, pWrtShell](sal_Int32 nResult) -> void {
314 if (nResult == RET_OK)
315 {
316 pObj->SetTitle(pDlg->GetTitle());
317 pObj->SetDescription(pDlg->GetDescription());
318 pObj->SetDecorative(pDlg->IsDecorative());
** CID 1608016: Uninitialized variables (UNINIT)
________________________________________________________________________________________________________
*** CID 1608016: Uninitialized variables (UNINIT)
/svx/source/fmcomp/fmgridif.cxx: 2219 in FmXGridPeer::removeGridControlListener(const com::sun::star::uno::Reference<com::sun::star::form::XGridControlListener> &)()
2213 }
2214
2215
2216 void SAL_CALL FmXGridPeer::removeGridControlListener( const Reference< XGridControlListener >& _listener )
2217 {
2218 std::unique_lock g(m_aMutex);
>>> CID 1608016: Uninitialized variables (UNINIT)
>>> Using uninitialized value "g._M_owns" when calling "removeInterface".
2219 m_aGridControlListeners.removeInterface( g, _listener );
2220 }
2221
2222
2223 sal_Int16 FmXGridPeer::getCurrentColumnPosition()
2224 {
** CID 1608015: (UNINIT)
________________________________________________________________________________________________________
*** CID 1608015: (UNINIT)
/animations/source/animcore/animcore.cxx: 1069 in animcore::<unnamed>::AnimationNode::setFillDefault(short)()
1063 void SAL_CALL AnimationNode::setFillDefault( sal_Int16 _filldefault )
1064 {
1065 std::unique_lock l( m_aMutex );
1066 if( _filldefault != mnFillDefault )
1067 {
1068 mnFillDefault = _filldefault;
>>> CID 1608015: (UNINIT)
>>> Using uninitialized value "l._M_owns" when calling "fireChangeListener".
1069 fireChangeListener(l);
1070 }
1071 }
1072
1073
1074 // XAnimationNode
/animations/source/animcore/animcore.cxx: 1069 in animcore::<unnamed>::AnimationNode::setFillDefault(short)()
1063 void SAL_CALL AnimationNode::setFillDefault( sal_Int16 _filldefault )
1064 {
1065 std::unique_lock l( m_aMutex );
1066 if( _filldefault != mnFillDefault )
1067 {
1068 mnFillDefault = _filldefault;
>>> CID 1608015: (UNINIT)
>>> Using uninitialized value "l._M_owns" when calling "fireChangeListener".
1069 fireChangeListener(l);
1070 }
1071 }
1072
1073
1074 // XAnimationNode
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu6VvXBlQRUbS683tC8265rGNPXqJ1ffcoLZCnTuJFQbNcTEkb4XaEQkzovKhJ5DB3c-3DhbES_A9M4dSy7guk8NP6DcfgslOyvJRzavztVIKj6nRqYjYpWom7SJFyX0y710bz0kUGtfVa1VhW3Bjqu3ofaIz0ANBZjj-2Fn-2FLqq8l8gkU3GGchE7-2BIV3umPmvY39HtGWcLG4VhhUGIgqQsxYfTA7U-2BZqCYsG0a-2Fnx5vi-2Bh5Ceo8kKfMbdbKEucjupFa-2BIU40myIKz6meuLm6Iuv1d90yea6kbb6h1smZQrjsvQ1UF7p3KTU-3D
More information about the LibreOffice
mailing list