[ARIES-1126] Weave hook could NPE

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: proxy-impl-1.0.2
• Component/s: Proxy
• Assignee: Daniel Kulp
• Reporter: Daniel Kulp
• Created: 18/Oct/13 19:19
• Updated: 30/Oct/13 10:00
• Resolved: 18/Oct/13 19:36

Description

In some non-reproducible cases with Equinox (likely during a refresh, but not really sure), the call at the first line of weave of:

Bundle b = wovenClass.getBundleWiring().getBundle();

can NPE due to the bundle wiring being null. This will result in bundles not starting at all. There is no way to work around this.

Two part proposed fix:
1) Check for null there and if null, don't do the bundle check. Note: if we DO have to weave, we do still need the wiring for the loader later.

2) If the "enabled" flag is set to "none", don't register the hook at all. We know nothing will be weaved by the hook so don't bother.