You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by "Vladimir Koshelev (JIRA)" <ji...@apache.org> on 2016/03/29 17:34:25 UTC
[jira] [Created] (LUCENENET-558) Some possible null reference
exceptions in ListExtensions.cs
Vladimir Koshelev created LUCENENET-558:
-------------------------------------------
Summary: Some possible null reference exceptions in ListExtensions.cs
Key: LUCENENET-558
URL: https://issues.apache.org/jira/browse/LUCENENET-558
Project: Lucene.Net
Issue Type: Bug
Components: Lucene.Net Core
Reporter: Vladimir Koshelev
Fix For: Lucene.Net 3.6
https://github.com/apache/lucenenet/blob/master/src/Lucene.Net.Core/Support/ListExtensions.cs#L18
"lt.Add(item);" should be changed to "list.Add(item)"
----------------------------------------------------------------------
https://github.com/apache/lucenenet/blob/master/src/Lucene.Net.Core/Support/ListExtensions.cs#L82
"if (current == null && item == null)"
The if's condition means that list can have null elements.
However, at https://github.com/apache/lucenenet/blob/master/src/Lucene.Net.Core/Support/ListExtensions.cs#L85
"if (current.Equals(item))"
current.Equals() is invoked, thus if the list have a null element, null reference exception would be thrown here.
I suggest to change the condition to (current != null && current.Equals(item)).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)