re PR libgcj/35979 (JNI method NewStringUTF throws NPE when passed a NULL pointer)

	PR libgcj/35979:
	* jni.cc (_Jv_JNI_NewStringUTF): Return NULL if bytes==NULL.

From-SVN: r134471

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index 7f501bd04c4f0ca6acbf367d74cb2f5d0006229f..c639181c878a51e662c0cf092e634f7495d30a74 100644 (file)
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2008-04-19  Tom Tromey  <tromey@redhat.com>
+
+       PR libgcj/35979:
+       * jni.cc (_Jv_JNI_NewStringUTF): Return NULL if bytes==NULL.
+
 2008-04-18  Paolo Bonzini  <bonzini@gnu.org>
 
        PR bootstrap/35457

diff --git a/libjava/jni.cc b/libjava/jni.cc
index 8bb2e59188130bf3fd0337d9533e7b7e05aa74dd..220046187b4e170ab9d97bb96f184a4d4e5b43c8 100644 (file)
--- a/libjava/jni.cc
+++ b/libjava/jni.cc
@@ -1,6 +1,6 @@
 // jni.cc - JNI implementation, including the jump table.
 
-/* Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007
+/* Copyright (C) 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008
    Free Software Foundation
 
    This file is part of libgcj.
@@ -1332,6 +1332,9 @@ _Jv_JNI_NewStringUTF (JNIEnv *env, const char *bytes)
 {
   try
     {
+      // For compatibility with the JDK.
+      if (!bytes)
+       return NULL;
       jstring result = JvNewStringUTF (bytes);
       return (jstring) wrap_value (env, result);
     }