Bug 67599 - NPE when cancelling search
Summary: NPE when cancelling search
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 RC3   Edit
Assignee: Jerome Lanneluc CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-17 04:43 EDT by Thomas M??der CLA
Modified: 2004-06-18 11:35 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas M??der CLA 2004-06-17 04:43:52 EDT 
I2004-06-16-20:00

1) create a workspace with JUnit 3.8.1 and org.eclipse.releng as source.
2) search for declarations of type *e* via the search dialog
3) cancel the search
4) observe. you get an exception:

Thread [Worker-4] (Suspended (exception NullPointerException))
	PossibleMatch.cleanUp() line: 46
	MatchLocator.locateMatches(JavaProject, PossibleMatch[], int, int) line: 719
	MatchLocator.locateMatches(JavaProject, PossibleMatchSet) line: 776
	MatchLocator.locateMatches(SearchDocument[]) line: 879
	JavaSearchParticipant.locateMatches(SearchDocument[], SearchPattern,
IJavaSearchScope, SearchRequestor, IProgressMonitor) line: 94
	SearchEngine.findMatches(SearchPattern, SearchParticipant[], IJavaSearchScope,
SearchRequestor, IProgressMonitor) line: 468
	SearchEngine.search(SearchPattern, SearchParticipant[], IJavaSearchScope,
SearchRequestor, IProgressMonitor) line: 757
	JavaSearchQuery.run(IProgressMonitor) line: 125
	InternalSearchUI$InternalSearchJob.run(IProgressMonitor) line: 83
	Worker.run() line: 66
Comment 1 Jerome Lanneluc CLA 2004-06-17 06:04:25 EDT 
Problem introduced while fixing bug 65831. Should check for null in the cleanup
() method.
Comment 2 Dirk Baeumer CLA 2004-06-17 08:43:22 EDT 
Is a regrssion to RC2. Approved by Dirk.
Comment 3 Philipe Mulet CLA 2004-06-17 09:08:40 EDT 
+1
Comment 4 Philipe Mulet CLA 2004-06-17 09:09:09 EDT 
Change reviewed.
Comment 5 Jerome Lanneluc CLA 2004-06-17 09:27:58 EDT 
Null check in PossibleMatch.cleanUp() released.
Comment 6 David Audel CLA 2004-06-18 11:35:57 EDT 
Verified for 3.0RC3 I200406180800