New Defects reported by Coverity Scan for LibreOffice

[scan-admin at coverity.com]

Hi,

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

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


** CID 1356339:  Uninitialized members  (UNINIT_CTOR)
/sc/source/ui/inc/checklistmenu.hxx: 254 in ScSearchEdit::ScSearchEdit(vcl::Window *)()


________________________________________________________________________________________________________
*** CID 1356339:  Uninitialized members  (UNINIT_CTOR)
/sc/source/ui/inc/checklistmenu.hxx: 254 in ScSearchEdit::ScSearchEdit(vcl::Window *)()
248     
249     class ScSearchEdit : public Edit
250     {
251     private:
252         ScTabStops*         mpTabStops;
253     public:
>>>     CID 1356339:  Uninitialized members  (UNINIT_CTOR)
>>>     Non-static class member "mpTabStops" is not initialized in this constructor nor in any functions that it calls.
254         ScSearchEdit(Window* pParent) : Edit(pParent) {}
255         virtual ~ScSearchEdit() {}
256     
257         virtual void MouseButtonDown( const MouseEvent& rMEvt ) override;
258         void SetTabStopsContainer( ScTabStops* pTabStops )  { mpTabStops = pTabStops; }
259     };

** CID 1356338:    (RESOURCE_LEAK)
/solenv/bin/concat-deps.c: 1097 in process()
/solenv/bin/concat-deps.c: 1097 in process()
/solenv/bin/concat-deps.c: 1097 in process()


________________________________________________________________________________________________________
*** CID 1356338:    (RESOURCE_LEAK)
/solenv/bin/concat-deps.c: 1097 in process()
1091         /* Note: yes we are going to leak 'buffer'
1092          * this is on purpose, to avoid cloning the 'key' out of it and our special
1093          * 'hash' just store the pointer to the key inside of buffer, hence it need
1094          * to remain allocated
1095          * coverity[leaked_storage] - this is on purpose
1096          */
>>>     CID 1356338:    (RESOURCE_LEAK)
>>>     Variable "buffer" going out of scope leaks the storage it points to.
1097         return rc;
1098     }
1099     
1100     static void usage(void)
1101     {
1102         fputs("Usage: concat-deps <file that contains dep_files>\n", stderr);
/solenv/bin/concat-deps.c: 1097 in process()
1091         /* Note: yes we are going to leak 'buffer'
1092          * this is on purpose, to avoid cloning the 'key' out of it and our special
1093          * 'hash' just store the pointer to the key inside of buffer, hence it need
1094          * to remain allocated
1095          * coverity[leaked_storage] - this is on purpose
1096          */
>>>     CID 1356338:    (RESOURCE_LEAK)
>>>     Variable "buffer" going out of scope leaks the storage it points to.
1097         return rc;
1098     }
1099     
1100     static void usage(void)
1101     {
1102         fputs("Usage: concat-deps <file that contains dep_files>\n", stderr);
/solenv/bin/concat-deps.c: 1097 in process()
1091         /* Note: yes we are going to leak 'buffer'
1092          * this is on purpose, to avoid cloning the 'key' out of it and our special
1093          * 'hash' just store the pointer to the key inside of buffer, hence it need
1094          * to remain allocated
1095          * coverity[leaked_storage] - this is on purpose
1096          */
>>>     CID 1356338:    (RESOURCE_LEAK)
>>>     Variable "buffer" going out of scope leaks the storage it points to.
1097         return rc;
1098     }
1099     
1100     static void usage(void)
1101     {
1102         fputs("Usage: concat-deps <file that contains dep_files>\n", stderr);

** CID 1356337:  Resource leaks  (RESOURCE_LEAK)
/sd/source/core/drawdoc3.cxx: 270 in SdDrawDocument::OpenBookmarkDoc(const rtl::OUString &)()


________________________________________________________________________________________________________
*** CID 1356337:  Resource leaks  (RESOURCE_LEAK)
/sd/source/core/drawdoc3.cxx: 270 in SdDrawDocument::OpenBookmarkDoc(const rtl::OUString &)()
264         SdDrawDocument* pBookmarkDoc = nullptr;
265     
266         if (!rBookmarkFile.isEmpty() && maBookmarkFile != rBookmarkFile)
267         {
268             SfxMedium* pMedium = new SfxMedium( rBookmarkFile, StreamMode::READ );
269             pBookmarkDoc = OpenBookmarkDoc(*pMedium);
>>>     CID 1356337:  Resource leaks  (RESOURCE_LEAK)
>>>     Variable "pMedium" going out of scope leaks the storage it points to.
270         }
271         else if (mxBookmarkDocShRef.Is())
272         {
273             pBookmarkDoc = mxBookmarkDocShRef->GetDoc();
274         }
275     


________________________________________________________________________________________________________
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