lucenenet-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vladimir Koshelev (JIRA)" <j...@apache.org>
Subject [jira] [Created] (LUCENENET-558) Some possible null reference exceptions in ListExtensions.cs
Date Tue, 29 Mar 2016 15:34:25 GMT
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)

Mime
View raw message