[JIRA] (JENKINS-57562) EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

o.v.nenashev@gmail.com (JIRA)

unread,

May 20, 2019, 10:29:02 AM5/20/19

to jenkinsc...@googlegroups.com

Oleg Nenashev created an issue

Jenkins /

JENKINS-57562

EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

Issue Type:	Improvement
Assignee:	FABRIZIO MANFREDI
Components:	ec2-plugin
Created:	2019-05-20 14:28
Labels:	jcasc-compatibility newbie-friendly
Priority:	Minor
Reporter:	Oleg Nenashev

It is a minor issue in the EC2 Plugin implementation.
instanceCapStr in https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.

I consider it is as an important issue, because the field is not exported by default

Add Comment

This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)

o.v.nenashev@gmail.com (JIRA)

unread,

May 20, 2019, 10:30:02 AM5/20/19

to jenkinsc...@googlegroups.com

Oleg Nenashev updated an issue

Jenkins /

JENKINS-57562

EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

Change By:	Oleg Nenashev

It is a minor issue in the EC2 Plugin implementation.
instanceCapStr in [https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java] is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.

I consider it is as an important issue, because the field is not exported by default .

The issue can be fixed by a small patch to properly handle nulls in the constructor arguments