Description of problem:

root/modules/common/ant-bundle/src/main/java/org/rhq/bundle/ant/type/SystemServiceType.java

The init method contains this:

if (this.configFile != null && !this.configFile.exists() || this.configFile.isDirectory()) {
            throw new BuildException("The 'configFile' attribute must be set to the path of an existing regular file.");
        }

Because of the lack of parenthesis, and the operator precendence rules of Java, this will not actually check that the config file is non-null.

Ie, if configFile is null, then this.configFile.isDirectory will still be called, resulting in a NullPointerException. I believe this should be changed to:

if (this.configFile != null && (!this.configFile.exists() || this.configFile.isDirectory())) {
            throw new BuildException("The 'configFile' attribute must be set to the path of an existing regular file.");
        }