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