GitHub user dilraj45 opened a pull request:
https://github.com/apache/commons-cli/pull/24
Fixes bugs in CommandLine.java which could possible lead to Null pointer
Explanation to code changes is given below
- [CommandLine.java
#L233](https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L233)<https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L233>
Method call to `getValuesList()` returns the possible values of `Option` for
which it is called as List or returns `null`. In case it returns `null`,
`addAll()` method call on type `List` with `null` as an argument will lead to a
NullPointerException. This can be prevented by explicitly checking and handling
the cases when it might return null. I have made changes for the same
- [CommandLine.java #L260
](https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L260)<https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L260>As
per the implementation of `stripLeadingHyphens(String)`, there exists cases
when it might return `null`. And calling `equals` method on a null return value
([#L263](https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L263))
will lead to null pointer exception. So as to prevent this I have made changes
to explicitly return null in case `stripLeadingHyphens` returns null
- [CommandLine.java
#L472](https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L472)
<https://github.com/apache/commons-cli/blob/b0024d482050a08efc36c3cabee37c0af0e57a10/src/main/java/org/apache/commons/cli/CommandLine.java#L472>
There is a possibility that `addOption` method is called with null as an
argument. In that case we will be adding null values to our list of processed
options (as `List` does not thrown any null pointer exception even if `add`
method is called with null values). So as to prevent this we need to make sure
that value that we are adding to list of processed option is not null.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dilraj45/commons-cli development
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/commons-cli/pull/24.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #24
----
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
