New Defects reported by Coverity Scan for LibreOffice

[Szymon Kłos]

Hi,

I will fix it.

Szymon

W dniu czw, 14.11.2019 o godzinie 10∶06 +0000, użytkownik Caolán
McNamara napisał:
> On Thu, 2019-11-14 at 08:10 +0000, scan-admin at coverity.com wrote:
> > Hi,
> > 
> > Please find the latest report on new defect(s) introduced to
> > LibreOffice found with Coverity Scan.
> 
> The below is new since...
> 
> commit a3c7a8282ddd08c7ed4a15d23089d09e418f8fae
> Author: Szymon Kłos <szymon.klos at collabora.com>
> Date:   Fri Nov 8 17:13:05 2019 +0100
> 
>     jsdialogs: apply .uno:Color and CharBackColor to floating text
> frames
> 
> I'm not sure what the right fix should be.
> 
> > 2 new defect(s) introduced to LibreOffice found with Coverity Scan.
> > 5 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 2 of 2 defect(s)
> > 
> > 
> > ** CID 1455691:  Null pointer dereferences  (REVERSE_INULL)
> > /sw/source/uibase/shells/drwtxtex.cxx: 660 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 
> > 
> > ___________________________________________________________________
> > __
> > ___________________________________
> > *** CID 1455691:  Null pointer dereferences  (REVERSE_INULL)
> > /sw/source/uibase/shells/drwtxtex.cxx: 660 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 654                 return;
> > 655         }
> > 656     
> > 657         std::unique_ptr<SfxItemSet> pNewArgs = pNewAttrs-
> > > Clone();
> > 
> > 658         lcl_convertStringArguments(nSlot, pNewArgs);
> > 659     
> > > > >     CID 1455691:  Null pointer dereferences  (REVERSE_INULL)
> > > > >     Null-checking "pNewAttrs" suggests that it may be null,
> > > > > but
> > > > > it has already been dereferenced on all paths leading to the
> > > > > check.
> > 
> > 660         if(nEEWhich && pNewAttrs)
> > 661         {
> > 662             std::unique_ptr<SfxPoolItem> pNewItem(pNewArgs-
> > > Get(nWhich).CloneSetWhich(nEEWhich));
> > 
> > 663             pNewArgs->Put(*pNewItem);
> > 664         }
> > 665     
> > 
> > ** CID 1455690:    (FORWARD_NULL)
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 
> > 
> > ___________________________________________________________________
> > __
> > ___________________________________
> > *** CID 1455690:    (FORWARD_NULL)
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 651     
> > 652             default:
> > 653                 assert(false && "wrong dispatcher");
> > 654                 return;
> > 655         }
> > 656     
> > > > >     CID 1455690:    (FORWARD_NULL)
> > > > >     Passing null pointer "pNewAttrs" to "Clone", which
> > > > > dereferences it. (The dereference happens because this is a
> > > > > virtual function call.)
> > 
> > 657         std::unique_ptr<SfxItemSet> pNewArgs = pNewAttrs-
> > > Clone();
> > 
> > 658         lcl_convertStringArguments(nSlot, pNewArgs);
> > 659     
> > 660         if(nEEWhich && pNewAttrs)
> > 661         {
> > 662             std::unique_ptr<SfxPoolItem> pNewItem(pNewArgs-
> > > Get(nWhich).CloneSetWhich(nEEWhich));
> > 
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 651     
> > 652             default:
> > 653                 assert(false && "wrong dispatcher");
> > 654                 return;
> > 655         }
> > 656     
> > > > >     CID 1455690:    (FORWARD_NULL)
> > > > >     Passing null pointer "pNewAttrs" to "Clone", which
> > > > > dereferences it. (The dereference happens because this is a
> > > > > virtual function call.)
> > 
> > 657         std::unique_ptr<SfxItemSet> pNewArgs = pNewAttrs-
> > > Clone();
> > 
> > 658         lcl_convertStringArguments(nSlot, pNewArgs);
> > 659     
> > 660         if(nEEWhich && pNewAttrs)
> > 661         {
> > 662             std::unique_ptr<SfxPoolItem> pNewItem(pNewArgs-
> > > Get(nWhich).CloneSetWhich(nEEWhich));
> > 
> > /sw/source/uibase/shells/drwtxtex.cxx: 657 in
> > SwDrawTextShell::Execute(SfxRequest &)()
> > 651     
> > 652             default:
> > 653                 assert(false && "wrong dispatcher");
> > 654                 return;
> > 655         }
> > 656     
> > > > >     CID 1455690:    (FORWARD_NULL)
> > > > >     Passing null pointer "pNewAttrs" to "Clone", which
> > > > > dereferences it. (The dereference happens because this is a
> > > > > virtual function call.)
> > 
> > 657         std::unique_ptr<SfxItemSet> pNewArgs = pNewAttrs-
> > > Clone();
> > 
> > 658         lcl_convertStringArguments(nSlot, pNewArgs);
> > 659     
> > 660         if(nEEWhich && pNewAttrs)
> > 661         {
> > 662             std::unique_ptr<SfxPoolItem> pNewItem(pNewArgs-
> > > Get(nWhich).CloneSetWhich(nEEWhich));
> 
>