[
https://issues.apache.org/jira/browse/ARIES-1105?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Dominik Przybysz closed ARIES-1105.
-----------------------------------
Resolution: Won't Do
Application module is deprecated and removed from the main repository
> Insufficient null-checks
> ------------------------
>
> Key: ARIES-1105
> URL: https://issues.apache.org/jira/browse/ARIES-1105
> Project: Aries
> Issue Type: Bug
> Components: Application
> Reporter: Jens Kordowski
> Priority: Major
>
> Hi,
> org.apache.aries.application.impl.ApplicationMetadataImpl
> org.apache.aries.application.impl.DeploymentMetadataImpl
> show several null-check problems:
> * setup(Manifest) method:
> this.appScope = this.appSymbolicName + "_" + this.appVersion.toString();
> if (this.appSymbolicName == null || this.appVersion == null) {
> -> either running into a NPE in the first place or the check is unnecessary
> * DeploymentMetadataImpl(Map):
> if (map != null) {
> for (Map.Entry<String, String> entry : map.entrySet()) {
> attributes.putValue(entry.getKey(), entry.getValue());
> }
> }
> parseDeploymentContent(map.get(AppConstants.DEPLOYMENT_CONTENT),
> _deploymentContent);
> -> what if map is null
> * DeploymentMetadataImpl(AriesApplication, Set):
> if ((appContentRange == null) && (useBundleRange == null)){
> _provisionSharedContent.add(dp);
> } else if (appContentRange.matches(info.getVersion())) {
> _deploymentContent.add(dp);
> } else if (useBundleRange.matches(info.getVersion())) {
> _deployedUseBundleContent.add(dp);
> }
> -> what if only one variable is null
> Best regards,
> Jens
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
