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.

4 new defect(s) introduced to LibreOffice found with Coverity Scan.


New defect(s) Reported-by: Coverity Scan
Showing 4 of 4 defect(s)


** CID 1458160:  Null pointer dereferences  (REVERSE_INULL)
/sw/source/core/frmedt/fefly1.cxx: 948 in SwFEShell::InsertDrawObj(SdrObject &, const Point &)()


________________________________________________________________________________________________________
*** CID 1458160:  Null pointer dereferences  (REVERSE_INULL)
/sw/source/core/frmedt/fefly1.cxx: 948 in SwFEShell::InsertDrawObj(SdrObject &, const Point &)()
942         SwContact* pContact = static_cast<SwContact*>(rDrawObj.GetUserCall());
943         if ( pContact )
944         {
945             pContact->MoveObjToVisibleLayer( &rDrawObj );
946         }
947     
>>>     CID 1458160:  Null pointer dereferences  (REVERSE_INULL)
>>>     Null-checking "pFormat" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
948         if ( pFormat )
949         {
950             // select drawing object
951             Imp()->GetDrawView()->MarkObj( &rDrawObj, Imp()->GetPageView() );
952         }
953         else

** CID 1458159:  Null pointer dereferences  (FORWARD_NULL)
/qadevOOo/tests/java/ifc/lang/_XMultiComponentFactory.java: 113 in ifc.lang._XMultiComponentFactory._getAvailableServiceNames()()


________________________________________________________________________________________________________
*** CID 1458159:  Null pointer dereferences  (FORWARD_NULL)
/qadevOOo/tests/java/ifc/lang/_XMultiComponentFactory.java: 113 in ifc.lang._XMultiComponentFactory._getAvailableServiceNames()()
107             else { // if service names are given, ignore result
108                 String[]erg = oObj.getAvailableServiceNames();
109                 result = (erg != null);
110             }
111     
112             log.println("Available service names:");
>>>     CID 1458159:  Null pointer dereferences  (FORWARD_NULL)
>>>     Accessing length of null array "availableServiceNames".
113             for(int i = 0; i < availableServiceNames.length; i++) {
114                 log.println("   " + availableServiceNames[i]);
115             }
116     
117             tRes.tested("getAvailableServiceNames()", result);
118         }

** CID 1458158:  Null pointer dereferences  (REVERSE_INULL)
/qadevOOo/tests/java/ifc/bridge/_XBridgeFactory.java: 131 in ifc.bridge._XBridgeFactory._getExistingBridges()()


________________________________________________________________________________________________________
*** CID 1458158:  Null pointer dereferences  (REVERSE_INULL)
/qadevOOo/tests/java/ifc/bridge/_XBridgeFactory.java: 131 in ifc.bridge._XBridgeFactory._getExistingBridges()()
125             log.println("Existing bridges :") ;
126             for (int i = 0; i < bridges.length; i++)
127                 log.println("  " + bridges[i].getDescription()) ;
128     
129             if (bridges.length > 0) bridgeName = bridges[0].getName() ;
130     
>>>     CID 1458158:  Null pointer dereferences  (REVERSE_INULL)
>>>     Null-checking "bridges" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
131             tRes.tested("getExistingBridges()", bridges != null) ;
132         }
133     
134         /**
135         * First creates connection with StarOffice process, using environment
136         * property <code>'CONNECTION_STRING'</code>. Then create bridge with unique name

** CID 1458157:  Null pointer dereferences  (REVERSE_INULL)
/qadevOOo/tests/java/ifc/lang/_XMultiServiceFactory.java: 78 in ifc.lang._XMultiServiceFactory._getAvailableServiceNames()()


________________________________________________________________________________________________________
*** CID 1458157:  Null pointer dereferences  (REVERSE_INULL)
/qadevOOo/tests/java/ifc/lang/_XMultiServiceFactory.java: 78 in ifc.lang._XMultiServiceFactory._getAvailableServiceNames()()
72             services = oObj.getAvailableServiceNames();
73     
74             for (int i = 0; i < services.length; i++) {
75                 log.println("Service" + i + ": " + services[i]);
76             }
77     
>>>     CID 1458157:  Null pointer dereferences  (REVERSE_INULL)
>>>     Null-checking "services" suggests that it may be null, but it has already been dereferenced on all paths leading to the check.
78             tRes.tested("getAvailableServiceNames()", services != null);
79         }
80     
81         /**
82          * Test creates instance of the first service from names array
83          * get by <code>getAvailableServiceNames()</code>. If the array


________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/ls/click?upn=nJaKvJSIH-2FPAfmty-2BK5tYpPklAc1eEA-2F1zfUjH6teExViPHTTReBArhCRZ3BE4kCjKjDqn2Dq3ZyEbAvAs31gRpU3vMPHDnoSx68vDAWjNU-3DJqC__OTq2XUZbbipYjyLSo6GRo-2FpVxQ9OzkDINu9UTS-2FQhSdO0F0jQniitrGlNxDIzPJiuYkKGcOLJukdXI6-2BCMWrvtNxD-2FM-2FK7MjnQT3LT-2BbA5NG-2BqOM8MV1sUAnz-2FR82DXzIoTjG6NxrPV1qc6I2nuv8E1K4AsyigNSQnT4ileOkc0UOM7rDM2VX3XzpqzkIOKo-2FySeNYBDDhuKlG-2Fz2ZlzPDqwgoF0HS-2FZfaQL3tX4Yammqq6P6pYA4ZpJJsp7rxRU