You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Julian Leichert (JIRA)" <ji...@apache.org> on 2017/09/07 12:33:00 UTC

[jira] [Created] (OFBIZ-9684) [FB] Package org.apache.ofbiz.service.group

Julian Leichert created OFBIZ-9684:
--------------------------------------

             Summary: [FB] Package org.apache.ofbiz.service.group
                 Key: OFBIZ-9684
                 URL: https://issues.apache.org/jira/browse/OFBIZ-9684
             Project: OFBiz
          Issue Type: Sub-task
    Affects Versions: Trunk
            Reporter: Julian Leichert
            Priority: Minor


GroupModel.java:55, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
- RCN: Redundant nullcheck of org.apache.ofbiz.service.group.GroupModel.groupName, which is known to be non-null in new org.apache.ofbiz.service.group.GroupModel(Element)

This method contains a redundant check of a known non-null value against the constant null.

GroupServiceModel.java:107, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
- RCN: Redundant nullcheck of model, which is known to be non-null in org.apache.ofbiz.service.group.GroupServiceModel.invoke(ServiceDispatcher, String, Map)

This method contains a redundant check of a known non-null value against the constant null.

ServiceGroupReader.java:43, MS_SHOULD_BE_FINAL, Priorität: Hoch
- MS: org.apache.ofbiz.service.group.ServiceGroupReader.groupsCache isn't final but should be

This static field public but not final, and could be changed by malicious code or by accident from another package. The field could be made final to avoid this vulnerability.

ServiceGroupReader.java:78, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
- RCN: Redundant nullcheck of groupName, which is known to be non-null in org.apache.ofbiz.service.group.ServiceGroupReader.addGroupDefinitions(ResourceHandler)

This method contains a redundant check of a known non-null value against the constant null.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)