New Defects reported by Coverity Scan for LibreOffice

scan-admin at coverity.com scan-admin at coverity.com
Sat Mar 5 19:21:08 UTC 2016


Hi,

Please find the latest report on new defect(s) introduced to LibreOffice found with Coverity Scan.

4 new defect(s) introduced to LibreOffice found with Coverity Scan.
10 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 4 of 4 defect(s)


** CID 1355253:  Uninitialized members  (UNINIT_CTOR)
/sc/source/core/tool/interpr8.cxx: 139 in ScETSForecastCalculation::ScETSForecastCalculation(unsigned long, SvNumberFormatter *)()


________________________________________________________________________________________________________
*** CID 1355253:  Uninitialized members  (UNINIT_CTOR)
/sc/source/core/tool/interpr8.cxx: 139 in ScETSForecastCalculation::ScETSForecastCalculation(unsigned long, SvNumberFormatter *)()
133         mbInitialised = false;
134         mnMonthDay = 0;
135         mpBase     = nullptr;
136         mpTrend    = nullptr;
137         mpPerIdx   = nullptr;
138         mpForecast = nullptr;
>>>     CID 1355253:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "bEDS" is not initialized in this constructor nor in any functions that it calls.
139     }
140     
141     ScETSForecastCalculation::~ScETSForecastCalculation()
142     {
143         delete mpBase;
144         delete mpTrend;

** CID 1355252:  Uninitialized members  (UNINIT_CTOR)
/editeng/source/outliner/outliner.cxx: 1283 in Outliner::Outliner(SfxItemPool *, unsigned short)()


________________________________________________________________________________________________________
*** CID 1355252:  Uninitialized members  (UNINIT_CTOR)
/editeng/source/outliner/outliner.cxx: 1283 in Outliner::Outliner(SfxItemPool *, unsigned short)()
1277         pEditEngine->SetBeginMovingParagraphsHdl( LINK( this, Outliner, BeginMovingParagraphsHdl ) );
1278         pEditEngine->SetEndMovingParagraphsHdl( LINK( this, Outliner, EndMovingParagraphsHdl ) );
1279         pEditEngine->SetBeginPasteOrDropHdl( LINK( this, Outliner, BeginPasteOrDropHdl ) );
1280         pEditEngine->SetEndPasteOrDropHdl( LINK( this, Outliner, EndPasteOrDropHdl ) );
1281     
1282         Init( nMode );
>>>     CID 1355252:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "bIsExpanding" is not initialized in this constructor nor in any functions that it calls.
1283     }
1284     
1285     Outliner::~Outliner()
1286     {
1287     
1288         pParaList->Clear( true );

** CID 1355251:    (DELETE_ARRAY)


________________________________________________________________________________________________________
*** CID 1355251:    (DELETE_ARRAY)
/sc/source/core/tool/interpr8.cxx: 1313 in ScInterpreter::ScForecast_Ets(ScETSType)()
1307         if ( !aETSCalc.PreprocessDataRange( pMatX, pMatY, nSmplInPrd, bDataCompletion,
1308                                            nAggregation,
1309                                            ( eETSType != etsStatAdd && eETSType != etsStatMult ? pTMat : nullptr ),
1310                                            eETSType ) )
1311         {
1312             PushError( aETSCalc.GetError() );
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpBase" which is an array.  It should be deleted with operator delete[].
1313             return;
1314         }
1315     
1316         switch ( eETSType )
1317         {
1318             case etsAdd    :
/sc/source/core/tool/interpr8.cxx: 1376 in ScInterpreter::ScForecast_Ets(ScETSType)()
1370                     else
1371                         PushError( aETSCalc.GetError() );
1372                 }
1373                 break;
1374         }
1375     
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpBase" which is an array.  It should be deleted with operator delete[].
1376         return;
1377     }
1378     
/sc/source/core/tool/interpr8.cxx: 1313 in ScInterpreter::ScForecast_Ets(ScETSType)()
1307         if ( !aETSCalc.PreprocessDataRange( pMatX, pMatY, nSmplInPrd, bDataCompletion,
1308                                            nAggregation,
1309                                            ( eETSType != etsStatAdd && eETSType != etsStatMult ? pTMat : nullptr ),
1310                                            eETSType ) )
1311         {
1312             PushError( aETSCalc.GetError() );
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpForecast" which is an array.  It should be deleted with operator delete[].
1313             return;
1314         }
1315     
1316         switch ( eETSType )
1317         {
1318             case etsAdd    :
/sc/source/core/tool/interpr8.cxx: 1376 in ScInterpreter::ScForecast_Ets(ScETSType)()
1370                     else
1371                         PushError( aETSCalc.GetError() );
1372                 }
1373                 break;
1374         }
1375     
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpForecast" which is an array.  It should be deleted with operator delete[].
1376         return;
1377     }
1378     
/sc/source/core/tool/interpr8.cxx: 1313 in ScInterpreter::ScForecast_Ets(ScETSType)()
1307         if ( !aETSCalc.PreprocessDataRange( pMatX, pMatY, nSmplInPrd, bDataCompletion,
1308                                            nAggregation,
1309                                            ( eETSType != etsStatAdd && eETSType != etsStatMult ? pTMat : nullptr ),
1310                                            eETSType ) )
1311         {
1312             PushError( aETSCalc.GetError() );
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpPerIdx" which is an array.  It should be deleted with operator delete[].
1313             return;
1314         }
1315     
1316         switch ( eETSType )
1317         {
1318             case etsAdd    :
/sc/source/core/tool/interpr8.cxx: 1376 in ScInterpreter::ScForecast_Ets(ScETSType)()
1370                     else
1371                         PushError( aETSCalc.GetError() );
1372                 }
1373                 break;
1374         }
1375     
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpPerIdx" which is an array.  It should be deleted with operator delete[].
1376         return;
1377     }
1378     
/sc/source/core/tool/interpr8.cxx: 1313 in ScInterpreter::ScForecast_Ets(ScETSType)()
1307         if ( !aETSCalc.PreprocessDataRange( pMatX, pMatY, nSmplInPrd, bDataCompletion,
1308                                            nAggregation,
1309                                            ( eETSType != etsStatAdd && eETSType != etsStatMult ? pTMat : nullptr ),
1310                                            eETSType ) )
1311         {
1312             PushError( aETSCalc.GetError() );
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpTrend" which is an array.  It should be deleted with operator delete[].
1313             return;
1314         }
1315     
1316         switch ( eETSType )
1317         {
1318             case etsAdd    :
/sc/source/core/tool/interpr8.cxx: 1376 in ScInterpreter::ScForecast_Ets(ScETSType)()
1370                     else
1371                         PushError( aETSCalc.GetError() );
1372                 }
1373                 break;
1374         }
1375     
>>>     CID 1355251:    (DELETE_ARRAY)
>>>     "~ScETSForecastCalculation" uses delete on "aETSCalc.mpTrend" which is an array.  It should be deleted with operator delete[].
1376         return;
1377     }
1378     

** CID 1355250:    (DEADCODE)
/sc/source/filter/dif/difexp.cxx: 169 in ScFormatFilterPluginImpl::ScExportDif(SvStream &, ScDocument *, const ScRange &, unsigned short)()
/sc/source/filter/dif/difexp.cxx: 193 in ScFormatFilterPluginImpl::ScExportDif(SvStream &, ScDocument *, const ScRange &, unsigned short)()


________________________________________________________________________________________________________
*** CID 1355250:    (DEADCODE)
/sc/source/filter/dif/difexp.cxx: 169 in ScFormatFilterPluginImpl::ScExportDif(SvStream &, ScDocument *, const ScRange &, unsigned short)()
163                         aOS.appendAscii(pEmptyData);
164                     break;
165                     case CELLTYPE_VALUE:
166                         aOS.appendAscii(pNumData);
167                         if( bPlain )
168                         {
>>>     CID 1355250:    (DEADCODE)
>>>     Execution cannot reach the expression "aOS.append(rtl::OUString(rtl::math::doubleToUString(aCell.mfValue, rtl_math_StringFormat_G, 14, u'.', true)))" inside this statement: "aOS.append(rtl::OUString(rt...".
169                             aOS.append(
170                                 rtl::math::doubleToUString(
171                                     aCell.mfValue, rtl_math_StringFormat_G, 14, '.', true));
172                         }
173                         else
174                         {
/sc/source/filter/dif/difexp.cxx: 193 in ScFormatFilterPluginImpl::ScExportDif(SvStream &, ScDocument *, const ScRange &, unsigned short)()
187                             aOS.appendAscii(pNumDataERROR);
188                         else if (aCell.mpFormula->IsValue())
189                         {
190                             aOS.appendAscii(pNumData);
191                             if( bPlain )
192                             {
>>>     CID 1355250:    (DEADCODE)
>>>     Execution cannot reach this statement: "fVal = aCell.mpFormula->Get...".
193                                 fVal = aCell.mpFormula->GetValue();
194                                 aOS.append(
195                                     rtl::math::doubleToUString(
196                                         fVal, rtl_math_StringFormat_G, 14, '.', true));
197                             }
198                             else


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/libreoffice?tab=overview

To manage Coverity Scan email notifications for "libreoffice at lists.freedesktop.org", click https://scan.coverity.com/subscriptions/edit?email=libreoffice%40lists.freedesktop.org&token=d6481d718a775246b2340f282ebe5939



More information about the LibreOffice mailing list