Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 366405 - NPE on loading PDE Target Platform with DirectoryBundleContainer having jars with non-OSGi manifests
Summary: NPE on loading PDE Target Platform with DirectoryBundleContainer having jars ...
Status: VERIFIED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.2   Edit
Hardware: PC Linux
: P3 major (vote)
Target Milestone: 3.8 M5   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 368876 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-12-12 08:58 EST by Aleksander Bandelj CLA
Modified: 2012-01-20 05:35 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aleksander Bandelj CLA 2011-12-12 08:58:20 EST
Build Identifier: I20111209-1447

Having plain jars or jars with invalid manifests in some directory in target platform prevents loading of target platform.

DirectoryBundleContainer#resolveBundles seems suspicious (regression ?)

TargetBundle rb = new TargetBundle(files[i]);
if (rb != null) {
...

Reproducible: Always

Steps to Reproduce:
Add directory with non-osgi jar to target platform, reload
Comment 1 Aleksander Bandelj CLA 2011-12-12 09:03:47 EST
It works if I change the condition to:

if (rb.getBundleInfo().getSymbolicName() != null)
Comment 2 Dani Megert CLA 2012-01-17 06:26:10 EST
Just for the records, the NPE is:

!ENTRY org.eclipse.ui 4 4 2012-01-17 12:21:37.710
!MESSAGE An internal error has occurred.
!STACK 0
java.lang.NullPointerException
	at org.eclipse.jface.viewers.StyledString.append(StyledString.java:264)
	at org.eclipse.jface.viewers.StyledString.append(StyledString.java:172)
	at org.eclipse.pde.internal.ui.shared.target.StyledBundleLabelProvider.appendBundleInfo(StyledBundleLabelProvider.java:169)
	at org.eclipse.pde.internal.ui.shared.target.StyledBundleLabelProvider.getStyledString(StyledBundleLabelProvider.java:85)
	at org.eclipse.pde.internal.ui.shared.target.StyledBundleLabelProvider.getText(StyledBundleLabelProvider.java:293)
	at org.eclipse.jface.viewers.ViewerComparator.getLabel(ViewerComparator.java:143)
	at org.eclipse.jface.viewers.ViewerComparator.compare(ViewerComparator.java:128)
	at org.eclipse.pde.internal.ui.shared.target.TargetContentsGroup$4.compare(TargetContentsGroup.java:286)
	at org.eclipse.jface.viewers.ViewerComparator$1.compare(ViewerComparator.java:189)
	at java.util.Arrays.mergeSort(Arrays.java:1270)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.mergeSort(Arrays.java:1281)
	at java.util.Arrays.sort(Arrays.java:1210)
	at org.eclipse.jface.viewers.ViewerComparator.sort(ViewerComparator.java:187)
	at org.eclipse.jface.viewers.AbstractTreeViewer.getSortedChildren(AbstractTreeViewer.java:635)
	at org.eclipse.jface.viewers.AbstractTreeViewer$1.run(AbstractTreeViewer.java:812)
	at org.eclipse.swt.custom.BusyIndicator.showWhile(BusyIndicator.java:70)
	at org.eclipse.jface.viewers.AbstractTreeViewer.createChildren(AbstractTreeViewer.java:789)
	at org.eclipse.jface.viewers.TreeViewer.createChildren(TreeViewer.java:644)
	at org.eclipse.jface.viewers.AbstractTreeViewer.createChildren(AbstractTreeViewer.java:760)
	at org.eclipse.jface.viewers.AbstractTreeViewer.internalInitializeTree(AbstractTreeViewer.java:1540)
	at org.eclipse.jface.viewers.TreeViewer.internalInitializeTree(TreeViewer.java:833)
	at org.eclipse.jface.viewers.AbstractTreeViewer$5.run(AbstractTreeViewer.java:1524)
	at org.eclipse.jface.viewers.StructuredViewer.preservingSelection(StructuredViewer.java:1443)
	at org.eclipse.jface.viewers.TreeViewer.preservingSelection(TreeViewer.java:403)
	at org.eclipse.jface.viewers.StructuredViewer.preservingSelection(StructuredViewer.java:1404)
	at org.eclipse.jface.viewers.CheckboxTreeViewer.preservingSelection(CheckboxTreeViewer.java:416)
	at org.eclipse.jface.viewers.AbstractTreeViewer.inputChanged(AbstractTreeViewer.java:1517)
	at org.eclipse.jface.viewers.ContentViewer.setInput(ContentViewer.java:280)
	at org.eclipse.jface.viewers.StructuredViewer.setInput(StructuredViewer.java:1690)
	at org.eclipse.pde.internal.ui.shared.target.TargetContentsGroup.setInput(TargetContentsGroup.java:955)
	at org.eclipse.pde.internal.ui.wizards.target.TargetDefinitionContentPage$4.runInUIThread(TargetDefinitionContentPage.java:266)
	at org.eclipse.ui.progress.UIJob$1.run(UIJob.java:95)
	at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:35)
	at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:135)
	at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:4140)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3757)
	at org.eclipse.jface.window.Window.runEventLoop(Window.java:825)
	at org.eclipse.jface.window.Window.open(Window.java:801)
	at org.eclipse.pde.internal.ui.preferences.TargetPlatformPreferencePage.handleEdit(TargetPlatformPreferencePage.java:489)
	at org.eclipse.pde.internal.ui.preferences.TargetPlatformPreferencePage.access$4(TargetPlatformPreferencePage.java:483)
	at org.eclipse.pde.internal.ui.preferences.TargetPlatformPreferencePage$9.widgetSelected(TargetPlatformPreferencePage.java:321)
	at org.eclipse.swt.widgets.TypedListener.handleEvent(TypedListener.java:240)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1053)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4165)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3754)
	at org.eclipse.jface.window.Window.runEventLoop(Window.java:825)
	at org.eclipse.jface.window.Window.open(Window.java:801)
	at org.eclipse.ui.internal.dialogs.WorkbenchPreferenceDialog.open(WorkbenchPreferenceDialog.java:215)
	at org.eclipse.ui.internal.handlers.ShowPreferencePageHandler.execute(ShowPreferencePageHandler.java:54)
	at org.eclipse.ui.internal.handlers.HandlerProxy.execute(HandlerProxy.java:293)
	at org.eclipse.core.commands.Command.executeWithChecks(Command.java:476)
	at org.eclipse.core.commands.ParameterizedCommand.executeWithChecks(ParameterizedCommand.java:508)
	at org.eclipse.ui.internal.handlers.HandlerService.executeCommand(HandlerService.java:169)
	at org.eclipse.ui.internal.handlers.SlaveHandlerService.executeCommand(SlaveHandlerService.java:241)
	at org.eclipse.ui.internal.actions.CommandAction.runWithEvent(CommandAction.java:157)
	at org.eclipse.jface.action.ActionContributionItem.handleWidgetSelection(ActionContributionItem.java:584)
	at org.eclipse.jface.action.ActionContributionItem.access$2(ActionContributionItem.java:501)
	at org.eclipse.jface.action.ActionContributionItem$5.handleEvent(ActionContributionItem.java:411)
	at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:84)
	at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:1053)
	at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:4165)
	at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3754)
	at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:2701)
	at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:2665)
	at org.eclipse.ui.internal.Workbench.access$4(Workbench.java:2499)
	at org.eclipse.ui.internal.Workbench$7.run(Workbench.java:679)
	at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332)
	at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:668)
	at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149)
	at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:124)
	at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:110)
	at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:79)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:352)
	at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:179)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
	at java.lang.reflect.Method.invoke(Method.java:597)
	at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:624)
	at org.eclipse.equinox.launcher.Main.basicRun(Main.java:579)
	at org.eclipse.equinox.launcher.Main.run(Main.java:1433)
	at org.eclipse.equinox.launcher.Main.main(Main.java:1409)
	at org.eclipse.core.launcher.Main.main(Main.java:34)
Comment 3 Curtis Windatt CLA 2012-01-17 16:28:37 EST
*** Bug 368876 has been marked as a duplicate of this bug. ***
Comment 4 Curtis Windatt CLA 2012-01-18 12:51:50 EST
With changes in bug 368753 we shouldn't be adding bundles that don't have a symbolic name.  However, I have added code to the styled string writer to deal with any missing information in the bundle info.

http://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=17f55b06b9862628426d259639c21c7d67b89181
Comment 5 Dani Megert CLA 2012-01-20 05:35:10 EST
(In reply to comment #4)
> However, I have added code to the styled string writer to deal
> with any missing information in the bundle info.
> 
> http://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=17f55b06b9862628426d259639c21c7d67b89181


That shouldn't hurt, but FYI: I was only able to get the NPE using 3.8 M4 but not with e.g. N20120116-2000.