[MCHECKSTYLE-344] StringIndexOutOfBoundsException in RuleUtil

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.17
• Fix Version/s: 3.1.0
• Component/s: checkstyle:check
• Assignee: Guillaume Boué
• Reporter: fdvxxii
• Created: 17/Nov/17 11:05
• Updated: 18/Feb/19 13:33
• Resolved: 24/Feb/18 22:24

Description

RuleUtil has a bug at line 95:

		final int end = eventSrcName.lastIndexOf('.');
		eventSrcName = eventSrcName.substring(0, end);		

This code leads to a StringIndexOutOfBoundsException if the variable does not contain a period. I don't know if its a convention to have a period in that string, but either way the error message should me more expressive.

Issue Links

• is duplicated by: MCHECKSTYLE-354 Plugin throws exception on encountering rule: SeparatorWrapDot of Google checkstyle
• is duplicated by: MCHECKSTYLE-367 Regexp module fails when "id" property without a "." is provided
• links to: GitHub Pull Request #1