Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 3.2
-
Fix Version/s: 3.3
-
Component/s: ACE-Components, Framework, ICE-Components
-
Labels:None
-
Environment:All
-
Assignee Priority:P2
-
ICEsoft Forum Reference:
Description
DOMResponseWriter.writeText does a null reference check on the text parameter passed in, and accordingly throws a NPE. At first glance it might appear that it could simply ignore the lack of text, but looking at ResponseWriter.writeText's JavaDoc, it's clear that this contract is in place, and so DOMResponseWriter is obligated to do what it's doing.
So, the onus falls on the callers to not call this method with a null text parameter. From auditing the ICEfaces code-base, many unchecked uses exist throughout the ACE and compat component code. Some uses do employ null checking, but they are in the minority. Each use needs to be fixed.
So, the onus falls on the callers to not call this method with a null text parameter. From auditing the ICEfaces code-base, many unchecked uses exist throughout the ACE and compat component code. Some uses do employ null checking, but they are in the minority. Each use needs to be fixed.
Activity
- All
- Comments
- History
- Activity
- Remote Attachments
- Subversion
88 usages, but only 3 files needed changing. All other cases are for certain not null: string literals, JS calls, JSON builder output, already null check at or before writeText(), value assigned or returned would never be null, etc.
M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\autocompleteentry\AutoCompleteEntryRenderer.java#33583
M: C:\svn\ossrepo\icefaces3\trunk\icefaces\compat\core\src\main\java\com\icesoft\faces\renderkit\dom_html_basic\OutputTextRenderer.java#33583
M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\selectmenu\SelectMenuRenderer.java#33583