[ https://issues.apache.org/jira/browse/NET-185?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sebb resolved NET-185. ---------------------- Resolution: Fixed Added a check just in case > Possible NPE in Threader.java > ----------------------------- > > Key: NET-185 > URL: https://issues.apache.org/jira/browse/NET-185 > Project: Commons Net > Issue Type: Bug > Components: NNTP > Environment: Fortify > Reporter: Henri Yandell > Priority: Minor > > In Threader.gatherSubjects, it appears to be possible for tail.next = > c.child; to be null [line 356]: > 349 if (old.threadable == null && c.threadable == null) { > 350 // both dummies - merge them > 351 ThreadContainer tail; > 352 for (tail = old.child; > 353 tail != null && tail.next != null; > 354 tail = tail.next); > 355 > 356 tail.next = c.child; > It also seems kind of pointless, given that the subsequent line immediately > resets tail to c.child. Perhaps the ; is not intended there? -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira