[CSV-100] CSVParser: getHeaderMap throws NPE

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0
• Component/s: Parser
• Labels: csvparser

Environment

commons-cvs version is the git-clone of current trunk

• Assignee: Gary D. Gregory
• Reporter: Jeanette Winzenburg
• Created: 18/Sep/13 13:00
• Updated: 14/Aug/14 20:06
• Resolved: 18/Sep/13 14:58

Description

title nearly says it all

Given a CSVParser parser, the following line throws an NPE:

Map<String, Integer> header = parser.getHeaderMap();

Stacktrace:

Caused by: java.lang.NullPointerException
at java.util.HashMap.<init>(HashMap.java:318)
at java.util.LinkedHashMap.<init>(LinkedHashMap.java:212)
at org.apache.commons.csv.CSVParser.getHeaderMap(CSVParser.java:288)

happens if the format doesn't have a headerMap.

to fix, check if the parser's headerMap is null before trying to create the returned map:

public Map<String, Integer> getHeaderMap() {
    return this.headerMap != null ?
       new LinkedHashMap<String, Integer>(this.headerMap)
       : null;
}