Bug 198278

Summary: [importexport] Exporting a file passes a null monitor to the upload service
Product: [Tools] Target Management Reporter: Javier Montalvo Orús <javier.montalvoorus>
Component: RSEAssignee: David McKnight <dmcknigh>
Status: RESOLVED FIXED QA Contact: Martin Oberhuber <mober.at+eclipse>
Severity: normal    
Priority: P3    
Version: 2.0   
Target Milestone: 2.0.1   
Hardware: PC   
OS: Windows 2000   
Whiteboard:
Attachments:
Description Flags
Patch passing a NulProgressMonitor to the upload service mober.at+eclipse: iplog-

Description Javier Montalvo Orús CLA 2007-07-30 12:10:28 EDT
It can cause a NPE if the service doesn't expect null monitors such as bug 198182.
In this case it would be more sensible passing a NullProgressMonitor rather than null.
Comment 1 Javier Montalvo Orús CLA 2007-07-30 12:17:21 EDT
Created attachment 74944 [details]
Patch passing a NulProgressMonitor to the upload service

This patch solves the issue as it passes a NulProgressMonitor instead of null to the upload service. This is just a suggestion, but it works fine.
Comment 2 David McKnight CLA 2007-07-30 12:39:25 EDT
I've committed the patch to cvs.  Thanks.
Comment 3 Martin Oberhuber CLA 2012-05-23 16:50:25 EDT
Comment on attachment 74944 [details]
Patch passing a NulProgressMonitor to the upload service

iplog- since Javier was committer since Aug 2006 already:
http://dev.eclipse.org/mhonarc/lists/dsdp-tm-dev/msg00375.html