[jira] [Commented] (OFBIZ-11385) Possible NPE in DatabaseUtil.getColumnInfo(...)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-11385) Possible NPE in DatabaseUtil.getColumnInfo(...)

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-11385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17271550#comment-17271550 ]

ASF subversion and git services commented on OFBIZ-11385:
---------------------------------------------------------

Commit 17864a11e94f783338d04829e83f0d15cd713346 in ofbiz-framework's branch refs/heads/trunk from Benjamin Jugl
[ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=17864a1 ]

Fixed: Possible NPE in DatabaseUtil.getColumnInfo(...) (OFBIZ-11385)

> Possible NPE in DatabaseUtil.getColumnInfo(...)
> -----------------------------------------------
>
>                 Key: OFBIZ-11385
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11385
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework/webtools
>    Affects Versions: Trunk
>            Reporter: Benjamin Jugl
>            Assignee: Benjamin Jugl
>            Priority: Major
>         Attachments: OFBIZ-11385_Possible_NPE_in_DatabaseUtil_001.patch
>
>
> Right at the start DatabaseUtil.getColumnInfo(....) does an if comparison to 0 with the unchecked parameter tableNames.
> The way this method and especially the parameter is used in DatabaseUtil.induceModelFromDb(Collection<String> messages) is not secure to not provide null-values for this parameter.
> I suggest using a UtilValidate.isEmpty(...) instead of the unsave comparison.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)