cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rajani Karuturi <Rajani.Karut...@citrix.com>
Subject Re: New Defects reported by Coverity Scan for cloudstack
Date Tue, 03 Jun 2014 06:48:01 GMT
These are the approximate number of defects reported by coverity from the generated code.

> /awsapi/src/com/amazon/ec2 -> around 4300

> /awsapi/src/com/amazon/s3 -> around 350


total defects -> around 6500

~Rajani



On 02-Jun-2014, at 4:57 pm, Rajani Karuturi <Rajani.Karuturi@citrix.com> wrote:

> Hi Hugo,
> 
> awsapi-generated-code	is excluded  for the project but I still see issues reported in
them.
> For example for file src/com/amazon/ec2/DeleteTagsResponseType.java
> 
> 
> Can you check the file exclude pattern? I think .* is missing (awsapi/src/com/amazon/.*)
> 
> Fixing this might give us a better report as I see lot of them listed in these files.
> 
> 
> ~Rajani
> 
> 
> 
> On 29-Nov-2013, at 9:28 pm, Hugo Trippaers <trippie@gmail.com> wrote:
> 
>> FYI
>> 
>> Sent from my iPhone
>> 
>> Begin forwarded message:
>> 
>>> From: scan-admin@coverity.com
>>> Date: 29 november 2013 14:39:56 CET
>>> Subject: New Defects reported by Coverity Scan for cloudstack
>>> 
>>> 
>>> Hi,
>>> 
>>> 
>>> Please find the latest report on new defect(s) introduced to cloudstack found
with Coverity Scan.
>>> 
>>> Defect(s) Reported-by: Coverity Scan
>>> Showing 6 of 6 defect(s)
>>> 
>>> 
>>> ** CID 1116269:  Nesting level does not match indentation  (NESTING_INDENT_MISMATCH)
>>> /awsapi/src/com/cloud/bridge/service/controller/s3/ServiceProvider.java: 124
in com.cloud.bridge.service.controller.s3.ServiceProvider.getManagementHostId()()
>>> 
>>> ** CID 1133706:  Dereference after null check  (FORWARD_NULL)
>>> /server/src/com/cloud/vm/UserVmManagerImpl.java: 2803 in com.cloud.vm.UserVmManagerImpl$3.doInTransaction(com.cloud.utils.db.TransactionStatus)()
>>> 
>>> ** CID 1133705:  Resource leak on an exceptional path  (RESOURCE_LEAK)
>>> /server/src/com/cloud/server/ConfigurationServerImpl.java: 638 in com.cloud.server.ConfigurationServerImpl.updateSSLKeystore()()
>>> 
>>> ** CID 1133704:  SS: Unread field should be static  (FB.SS_SHOULD_BE_STATIC)
>>> /server/src/com/cloud/uuididentity/UUIDManagerImpl.java: 43 in ()
>>> 
>>> ** CID 1133703:  Dm: Dubious method used  (FB.DM_DEFAULT_ENCODING)
>>> /plugins/user-authenticators/ldap/src/org/apache/cloudstack/api/command/LdapImportUsersCmd.java:
197 in org.apache.cloudstack.api.command.LdapImportUsersCmd.generatePassword()()
>>> 
>>> ** CID 1133702:  DLS: Dead local store  (FB.DLS_DEAD_LOCAL_STORE)
>>> /plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/model/VirtualMachineModel.java:
119 in org.apache.cloudstack.network.contrail.model.VirtualMachineModel.buildServiceInstance(org.apache.cloudstack.network.contrail.model.ModelController,
java.lang.String)()
>>> 
>>> 
>>> 
>>> 
>>> ________________________________________________________________________________________________________
>>> To view the defects in Coverity Scan visit, http://scan.coverity.com
>>> 
>>> To unsubscribe from the email notification for new defects, http://scan5.coverity.com/cgi-bin/unsubscribe.py
>>> 
>>> 
>>> 
> 


Mime
View raw message