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
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
yip.ng
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #33583 | Fri Feb 22 15:52:40 MST 2013 | yip.ng | |
Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/autocompleteentry/AutoCompleteEntryRenderer.java
MODIFY /icefaces3/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/OutputTextRenderer.java MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/selectmenu/SelectMenuRenderer.java |
Ken Fyten
made changes -
Assignee | Mark Collette [ mark.collette ] | yip.ng [ yip.ng ] |
Assignee Priority | P1 [ 10010 ] | P2 [ 10011 ] |
Ken Fyten
made changes -
Assignee Priority | P1 [ 10010 ] |
Mark Collette
made changes -
Field | Original Value | New Value |
---|---|---|
Assignee | Mark Collette [ mark.collette ] |
Mark Collette
created issue -
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