[WICKET-5784] arraycopy with bad length in AbstractRequestLogger:172

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 6.17.0
• Fix Version/s: 7.0.0-M5, 6.19.0
• Component/s: wicket, wicket-devutils
• Labels: easyfix patch
• Environment: 6.17.0 with DebugBar from devutils on Tomcat 7, Java 7 (Oracle)
• Assignee: Sven Meier
• Reporter: Jan Cerny
• Created: 05/Dec/14 16:19
• Updated: 05/Dec/14 18:52
• Resolved: 05/Dec/14 18:52

Description

When clicking on DebugBar org.apache.wicket.devutils.inspector.LiveSessionsPage NullPointerException is thrown.
After investigating the reason I think AbstractRequestLogger:172 arraycopy params cause it.
arraycopy(requestWindow, 0, copy, requestWindow.length - oldestPos, indexInWindow);
Should be changed to:
arraycopy(requestWindow, 0, copy, requestWindow.length - oldestPos, oldestPos);