You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by Romain Manni-Bucau <rm...@gmail.com> on 2016/02/10 07:44:38 UTC

Fwd: tomee git commit: Resolve NPE

Hi Jon

repositories is only null when stop has been called normally so no need to
protect it in start.

Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber
<http://www.tomitribe.com>

---------- Forwarded message ----------
From: <jg...@apache.org>
Date: 2016-02-09 23:15 GMT+01:00
Subject: tomee git commit: Resolve NPE
To: commits@tomee.apache.org


Repository: tomee
Updated Branches:
  refs/heads/tomee-1.7.x 652642811 -> 82ecaa994


Resolve NPE


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/82ecaa99
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/82ecaa99
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/82ecaa99

Branch: refs/heads/tomee-1.7.x
Commit: 82ecaa99492fcfd8b758c48b434c714d89f097ca
Parents: 6526428
Author: Jonathan Gallimore <jo...@jrg.me.uk>
Authored: Tue Feb 9 22:15:01 2016 +0000
Committer: Jonathan Gallimore <jo...@jrg.me.uk>
Committed: Tue Feb 9 22:15:01 2016 +0000

----------------------------------------------------------------------
 .../java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/82ecaa99/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
----------------------------------------------------------------------
diff --git
a/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
b/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
index e8ffefe..0e87c72 100644
---
a/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
+++
b/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/LazyStopWebappClassLoader.java
@@ -254,6 +254,9 @@ public class LazyStopWebappClassLoader extends
WebappClassLoader {
     // will be in the webapp
     @Override
     public void start() throws LifecycleException {
+        if (this.repositories == null) {
+            this.repositories = new String[0];
+        }
         super.start(); // do it first otherwise we can't use this as
classloader

         // mainly for tomee-maven-plugin