New Defects reported by Coverity Scan for LibreOffice

scan-admin at coverity.com scan-admin at coverity.com
Fri Jun 12 01:32:15 PDT 2015


Hi,

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

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


** CID 1306215:  Uninitialized members  (UNINIT_CTOR)
/libreofficekit/source/gtk/tilebuffer.hxx: 54 in Tile::Tile()()


________________________________________________________________________________________________________
*** CID 1306215:  Uninitialized members  (UNINIT_CTOR)
/libreofficekit/source/gtk/tilebuffer.hxx: 54 in Tile::Tile()()
48        This class represents a single tile in the tile buffer.
49        It encloses a reference to GdkPixBuf containing the pixel data of the tile.
50     */
51     class Tile
52     {
53      public:
>>>     CID 1306215:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "m_pBuffer" is not initialized in this constructor nor in any functions that it calls.
54         Tile() : valid(false) {}
55         ~Tile() { }
56     
57         /**
58            Tells if this tile is valid or not. Initialised to 0 (invalid) during
59            object creation.

** CID 1306214:  Uninitialized members  (UNINIT_CTOR)
/vcl/qa/cppunit/lifecycle.cxx: 214 in LeakTestObject::LeakTestObject()()


________________________________________________________________________________________________________
*** CID 1306214:  Uninitialized members  (UNINIT_CTOR)
/vcl/qa/cppunit/lifecycle.cxx: 214 in LeakTestObject::LeakTestObject()()
208     
209     class LeakTestObject
210     {
211         bool                mbDeleted;
212         VclPtr<vcl::Window> mxRef;
213         void               *mpRef;
>>>     CID 1306214:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "mpRef" is not initialized in this constructor nor in any functions that it calls.
214         LeakTestObject() {}
215     public:
216         template<typename vcl_type, typename... Arg> static LeakTestObject *
217             Create(Arg &&... arg)
218         {
219             LeakTestObject *pNew = new LeakTestObject();

** CID 1306212:  Program hangs  (INFINITE_LOOP)
/vcl/qa/cppunit/timer.cxx: 122 in TimerTest::testIdleMainloop()()


________________________________________________________________________________________________________
*** CID 1306212:  Program hangs  (INFINITE_LOOP)
/vcl/qa/cppunit/timer.cxx: 122 in TimerTest::testIdleMainloop()()
116     
117     // tdf#91727
118     void TimerTest::testIdleMainloop()
119     {
120         bool bTriggered = false;
121         IdleBool aTest( bTriggered );
>>>     CID 1306212:  Program hangs  (INFINITE_LOOP)
>>>     If "bTriggered" is initially false then it will remain false.
122         while (!bTriggered)
123         {
124             ImplSVData* pSVData = ImplGetSVData();
125     
126             // can't test this via Application::Yield since this
127             // also processes all tasks directly via the scheduler.

** CID 1306211:  Control flow issues  (DEADCODE)
/connectivity/source/drivers/file/fanalyzer.cxx: 134 in connectivity::file::OSQLAnalyzer::bindRow(std::vector<connectivity::file::OCode *, std::allocator<connectivity::file::OCode *>> &, const rtl::Reference<connectivity::OValueRefVector> &, std::list<std::map<int, int, std::less<int>, std::allocator<std::pair<const int, int>>> *, std::allocator<std::map<int, int, std::less<int>, std::allocator<std::pair<const int, int>>> *>> &)()


________________________________________________________________________________________________________
*** CID 1306211:  Control flow issues  (DEADCODE)
/connectivity/source/drivers/file/fanalyzer.cxx: 134 in connectivity::file::OSQLAnalyzer::bindRow(std::vector<connectivity::file::OCode *, std::allocator<connectivity::file::OCode *>> &, const rtl::Reference<connectivity::OValueRefVector> &, std::list<std::map<int, int, std::less<int>, std::allocator<std::pair<const int, int>>> *, std::allocator<std::map<int, int, std::less<int>, std::allocator<std::pair<const int, int>>> *>> &)()
128         {
129             OOperandAttr* pAttr = PTR_CAST(OOperandAttr,(*aIter));
130             if (pAttr)
131             {
132                 if (pEvaluateSet)
133                 {
>>>     CID 1306211:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach this statement: "_rEvaluateSetList->push_bac...".
134                     _rEvaluateSetList.push_back(pEvaluateSet);
135                     pEvaluateSet = NULL;
136                 }
137                 pAttr->bindValue(_pRow);
138             }
139         }

** CID 1269539:    (RESOURCE_LEAK)
/usr/include/boost/iostreams/chain.hpp: 259 in boost::iostreams::detail::chain_base<boost::iostreams::chain<boost::iostreams::output, char, std::char_traits<char>, std::allocator<char>>, char, std::char_traits<char>, std::allocator<char>, boost::iostreams::output>::push_impl<boost::iostreams::basic_gzip_decompressor<std::allocator<char>>>(const T1 &, long, long)()
/usr/include/boost/iostreams/chain.hpp: 259 in boost::iostreams::detail::chain_base<boost::iostreams::chain<boost::iostreams::output, char, std::char_traits<char>, std::allocator<char>>, char, std::char_traits<char>, std::allocator<char>, boost::iostreams::output>::push_impl<boost::iostreams::back_insert_device<std::basic_string<char, std::char_traits<char>, std::allocator<char>>>>(const T1 &, long, long)()


________________________________________________________________________________________________________
*** CID 1269539:    (RESOURCE_LEAK)
/usr/include/boost/iostreams/chain.hpp: 259 in boost::iostreams::detail::chain_base<boost::iostreams::chain<boost::iostreams::output, char, std::char_traits<char>, std::allocator<char>>, char, std::char_traits<char>, std::allocator<char>, boost::iostreams::output>::push_impl<boost::iostreams::basic_gzip_decompressor<std::allocator<char>>>(const T1 &, long, long)()
253                 pback_size != -1 ?
254                     pback_size :
255                     pimpl_->pback_size_;
256             std::auto_ptr<streambuf_t>
257                 buf(new streambuf_t(t, buffer_size, pback_size));
258             list().push_back(buf.get());
>>>     CID 1269539:    (RESOURCE_LEAK)
>>>     Ignoring storage allocated by "buf.release()" leaks it.
259             buf.release();
260             if (is_device<component_type>::value) {
261                 pimpl_->flags_ |= f_complete | f_open;
262                 for ( iterator first = list().begin(),
263                                last = list().end();
264                       first != last;
/usr/include/boost/iostreams/chain.hpp: 259 in boost::iostreams::detail::chain_base<boost::iostreams::chain<boost::iostreams::output, char, std::char_traits<char>, std::allocator<char>>, char, std::char_traits<char>, std::allocator<char>, boost::iostreams::output>::push_impl<boost::iostreams::back_insert_device<std::basic_string<char, std::char_traits<char>, std::allocator<char>>>>(const T1 &, long, long)()
253                 pback_size != -1 ?
254                     pback_size :
255                     pimpl_->pback_size_;
256             std::auto_ptr<streambuf_t>
257                 buf(new streambuf_t(t, buffer_size, pback_size));
258             list().push_back(buf.get());
>>>     CID 1269539:    (RESOURCE_LEAK)
>>>     Ignoring storage allocated by "buf.release()" leaks it.
259             buf.release();
260             if (is_device<component_type>::value) {
261                 pimpl_->flags_ |= f_complete | f_open;
262                 for ( iterator first = list().begin(),
263                                last = list().end();
264                       first != last;


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://scan.coverity.com/projects/211?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