Mantis - Resin
Viewing Issue Advanced Details
ID: Category: Severity: Reproducibility: Date Submitted: Last Update:
3236 trivial always 01-09-09 18:46 01-12-09 16:39
Reporter: koreth Platform:  
Assigned To: ferg OS:  
Priority: normal OS Version:  
Status: closed Product Version: 4.0.0  
Product Build: Resolution: fixed  
Projection: none      
ETA: none Fixed in Version: 4.0.0  
Summary: 0003236: Patch: Get rid of some unnecessary NPEs
Description: When I run under a debugger and break on NPEs, it's annoying to get lots of false hits from OsgiManager. So here's a patch to get rid of its NPEs. (Whether the fact that I'm getting them is a symptom of something else, I can't say, but from the structure of the code it looks like exceptions are totally expected here.)

--- a/modules/kernel/src/com/caucho/osgi/OsgiManager.java
+++ b/modules/kernel/src/com/caucho/osgi/OsgiManager.java
@@ -349,9 +349,11 @@ public class OsgiManager
     synchronized (_exportMap) {
       loader = _publishedExportMap.get(packageName);
 
- try {
- return loader.getResource(name);
- } catch (Exception e) {
+ if (loader != null) {
+ try {
+ return loader.getResource(name);
+ } catch (Exception e) {
+ }
       }
     }
Steps To Reproduce:
Additional Information:
Relationships
Attached Files:

There are no notes attached to this issue.