You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Oliver Heger (JIRA)" <ji...@apache.org> on 2016/03/24 21:13:26 UTC
[jira] [Updated] (CONFIGURATION-421)
VFSFileChangedReloadingStrategy.init() uses configuration before checking
it for null
[ https://issues.apache.org/jira/browse/CONFIGURATION-421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Oliver Heger updated CONFIGURATION-421:
---------------------------------------
Fix Version/s: (was: 2.0)
2.x
> VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null
> -------------------------------------------------------------------------------------
>
> Key: CONFIGURATION-421
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-421
> Project: Commons Configuration
> Issue Type: Bug
> Components: File reloading
> Reporter: Sebb
> Fix For: 2.x
>
> Attachments: VFSFCRS.init - check for null before using configuration.diff
>
>
> VFSFileChangedReloadingStrategy.init() uses configuration before checking it for null:
> {noformat}
> if (configuration.getURL() == null && configuration.getFileName() == null)
> {
> return;
> }
> if (this.configuration == null)
> {
> throw new IllegalStateException("No configuration has been set for this strategy");
> }
> {noformat}
> The if statements should be swapped.
> [Note found by Findbugs]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)