You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ff...@apache.org on 2013/05/16 04:25:40 UTC
svn commit: r1483154 - in /cxf/branches/2.7.x-fixes: ./
maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/
maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/
maven-plugins/java2ws-plugin/src/main/jav...
Author: ffang
Date: Thu May 16 02:25:39 2013
New Revision: 1483154
URL: http://svn.apache.org/r1483154
Log:
Merged revisions 1483151 via svnmerge from
https://svn.apache.org/repos/asf/cxf/trunk
........
r1483151 | ffang | 2013-05-16 10:03:10 +0800 (四, 16 5 2013) | 1 line
[CXF-5020]add NPE guard when we restore original java.class.path system property
........
Modified:
cxf/branches/2.7.x-fixes/ (props changed)
cxf/branches/2.7.x-fixes/maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java
cxf/branches/2.7.x-fixes/maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/WSDLToIDLPlugin.java
cxf/branches/2.7.x-fixes/maven-plugins/java2ws-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java
cxf/branches/2.7.x-fixes/maven-plugins/wadl2java-plugin/src/main/java/org/apache/cxf/maven_plugin/common/ClassLoaderSwitcher.java
Propchange: cxf/branches/2.7.x-fixes/
------------------------------------------------------------------------------
Merged /cxf/trunk:r1483151
Propchange: cxf/branches/2.7.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.
Modified: cxf/branches/2.7.x-fixes/maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java?rev=1483154&r1=1483153&r2=1483154&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java (original)
+++ cxf/branches/2.7.x-fixes/maven-plugins/codegen-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java Thu May 16 02:25:39 2013
@@ -118,7 +118,9 @@ public class ClassLoaderSwitcher {
*/
public void restoreClassLoader() {
Thread.currentThread().setContextClassLoader(origContextClassloader);
- System.setProperty("java.class.path", origClassPath);
+ if (origClassPath != null) {
+ System.setProperty("java.class.path", origClassPath);
+ }
Map<Object, Object> newProps = new HashMap<Object, Object>(System.getProperties());
for (Object o : newProps.keySet()) {
Modified: cxf/branches/2.7.x-fixes/maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/WSDLToIDLPlugin.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/WSDLToIDLPlugin.java?rev=1483154&r1=1483153&r2=1483154&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/WSDLToIDLPlugin.java (original)
+++ cxf/branches/2.7.x-fixes/maven-plugins/corba/src/main/java/org/apache/cxf/maven_plugin/corba/maven/plugins/WSDLToIDLPlugin.java Thu May 16 02:25:39 2013
@@ -169,7 +169,9 @@ public class WSDLToIDLPlugin extends Abs
bus.shutdown(true);
}
Thread.currentThread().setContextClassLoader(origContext);
- System.setProperty("java.class.path", cp);
+ if (cp != null) {
+ System.setProperty("java.class.path", cp);
+ }
Map<Object, Object> newProps = new HashMap<Object, Object>(System.getProperties());
for (Object o : newProps.keySet()) {
Modified: cxf/branches/2.7.x-fixes/maven-plugins/java2ws-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/maven-plugins/java2ws-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java?rev=1483154&r1=1483153&r2=1483154&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/maven-plugins/java2ws-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java (original)
+++ cxf/branches/2.7.x-fixes/maven-plugins/java2ws-plugin/src/main/java/org/apache/cxf/maven_plugin/ClassLoaderSwitcher.java Thu May 16 02:25:39 2013
@@ -111,7 +111,9 @@ public class ClassLoaderSwitcher {
*/
public void restoreClassLoader() {
Thread.currentThread().setContextClassLoader(origContextClassloader);
- System.setProperty("java.class.path", origClassPath);
+ if (origClassPath != null) {
+ System.setProperty("java.class.path", origClassPath);
+ }
Map<Object, Object> newProps = new HashMap<Object, Object>(System.getProperties());
for (Object o : newProps.keySet()) {
Modified: cxf/branches/2.7.x-fixes/maven-plugins/wadl2java-plugin/src/main/java/org/apache/cxf/maven_plugin/common/ClassLoaderSwitcher.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.7.x-fixes/maven-plugins/wadl2java-plugin/src/main/java/org/apache/cxf/maven_plugin/common/ClassLoaderSwitcher.java?rev=1483154&r1=1483153&r2=1483154&view=diff
==============================================================================
--- cxf/branches/2.7.x-fixes/maven-plugins/wadl2java-plugin/src/main/java/org/apache/cxf/maven_plugin/common/ClassLoaderSwitcher.java (original)
+++ cxf/branches/2.7.x-fixes/maven-plugins/wadl2java-plugin/src/main/java/org/apache/cxf/maven_plugin/common/ClassLoaderSwitcher.java Thu May 16 02:25:39 2013
@@ -120,7 +120,9 @@ public class ClassLoaderSwitcher {
*/
public void restoreClassLoader() {
Thread.currentThread().setContextClassLoader(origContextClassloader);
- System.setProperty("java.class.path", origClassPath);
+ if (origClassPath != null) {
+ System.setProperty("java.class.path", origClassPath);
+ }
Map<Object, Object> newProps = new HashMap<Object, Object>(System.getProperties());
for (Object o : newProps.keySet()) {