public inbox for java-prs@sourceware.org
help / color / mirror / Atom feed
* [Bug libgcj/28178]  New: jniEnv->DeleteLocalRef (null) fails
@ 2006-06-27 14:03 hakan dot hjort at gmail dot com
  2006-06-27 14:37 ` [Bug libgcj/28178] " hakan dot hjort at gmail dot com
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: hakan dot hjort at gmail dot com @ 2006-06-27 14:03 UTC (permalink / raw)
  To: java-prs

According to Sun documentation NULL is a allowed to call DeleteLocalRef() with
a NULL argument.  The GCJ jni implementation exits (Aborted) in this case.

From: http://java.sun.com/docs/books/jni/html/exceptions.html
6.1.2 A Utility Function
Passing NULL to DeleteLocalRef is a no-op, which is an appropriate action if
FindClass fails and returns NULL.


-- 
           Summary: jniEnv->DeleteLocalRef (null) fails
           Product: gcc
           Version: 4.0.2
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libgcj
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hakan dot hjort at gmail dot com
GCC target triplet: i386-redhat-linux


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug libgcj/28178] jniEnv->DeleteLocalRef (null) fails
  2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
@ 2006-06-27 14:37 ` hakan dot hjort at gmail dot com
  2006-06-27 15:16 ` tromey at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: hakan dot hjort at gmail dot com @ 2006-06-27 14:37 UTC (permalink / raw)
  To: java-prs



------- Comment #1 from hakan dot hjort at gmail dot com  2006-06-27 14:37 -------
Created an attachment (id=11760)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=11760&action=view)
Testcase FindClass on bad class + DeleteLocalRef

Build as
gcc delete_null.c -lgcj
Should have exit status 0 and print "OK: exception pending".
I've tested it successfully using the Sun JVM.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug libgcj/28178] jniEnv->DeleteLocalRef (null) fails
  2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
  2006-06-27 14:37 ` [Bug libgcj/28178] " hakan dot hjort at gmail dot com
@ 2006-06-27 15:16 ` tromey at gcc dot gnu dot org
  2006-06-27 15:33 ` tromey at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: tromey at gcc dot gnu dot org @ 2006-06-27 15:16 UTC (permalink / raw)
  To: java-prs



-- 

tromey at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |tromey at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-06-27 15:16:01
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug libgcj/28178] jniEnv->DeleteLocalRef (null) fails
  2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
  2006-06-27 14:37 ` [Bug libgcj/28178] " hakan dot hjort at gmail dot com
  2006-06-27 15:16 ` tromey at gcc dot gnu dot org
@ 2006-06-27 15:33 ` tromey at gcc dot gnu dot org
  2006-06-27 15:42 ` tromey at gcc dot gnu dot org
  2006-06-27 15:46 ` tromey at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: tromey at gcc dot gnu dot org @ 2006-06-27 15:33 UTC (permalink / raw)
  To: java-prs



------- Comment #2 from tromey at gcc dot gnu dot org  2006-06-27 15:33 -------
Subject: Bug 28178

Author: tromey
Date: Tue Jun 27 15:33:24 2006
New Revision: 115034

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=115034
Log:
        PR libgcj/28178:
        * jni.cc (_Jv_JNI_DeleteLocalRef): Ignore null argument.
        (_Jv_JNI_DeleteGlobalRef): Likewise.
        * testsuite/libjava.jni/PR28178.java: New file.
        * testsuite/libjava.jni/PR28178.c: New file.
        * testsuite/libjava.jni/PR28178.out: New file.

Added:
    trunk/libjava/testsuite/libjava.jni/PR28178.c
    trunk/libjava/testsuite/libjava.jni/PR28178.java
    trunk/libjava/testsuite/libjava.jni/PR28178.out
Modified:
    trunk/libjava/ChangeLog
    trunk/libjava/jni.cc


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug libgcj/28178] jniEnv->DeleteLocalRef (null) fails
  2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
                   ` (2 preceding siblings ...)
  2006-06-27 15:33 ` tromey at gcc dot gnu dot org
@ 2006-06-27 15:42 ` tromey at gcc dot gnu dot org
  2006-06-27 15:46 ` tromey at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: tromey at gcc dot gnu dot org @ 2006-06-27 15:42 UTC (permalink / raw)
  To: java-prs



------- Comment #3 from tromey at gcc dot gnu dot org  2006-06-27 15:42 -------
Subject: Bug 28178

Author: tromey
Date: Tue Jun 27 15:42:22 2006
New Revision: 115035

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=115035
Log:
        PR libgcj/28178:
        * jni.cc (_Jv_JNI_DeleteLocalRef): Ignore null argument.
        (_Jv_JNI_DeleteGlobalRef): Likewise.
        * testsuite/libjava.jni/PR28178.java: New file.
        * testsuite/libjava.jni/PR28178.c: New file.
        * testsuite/libjava.jni/PR28178.out: New file.

Added:
    branches/gcc-4_1-branch/libjava/testsuite/libjava.jni/PR28178.c
      - copied unchanged from r115034,
trunk/libjava/testsuite/libjava.jni/PR28178.c
    branches/gcc-4_1-branch/libjava/testsuite/libjava.jni/PR28178.java
      - copied unchanged from r115034,
trunk/libjava/testsuite/libjava.jni/PR28178.java
    branches/gcc-4_1-branch/libjava/testsuite/libjava.jni/PR28178.out
      - copied unchanged from r115034,
trunk/libjava/testsuite/libjava.jni/PR28178.out
Modified:
    branches/gcc-4_1-branch/libjava/ChangeLog
    branches/gcc-4_1-branch/libjava/jni.cc


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug libgcj/28178] jniEnv->DeleteLocalRef (null) fails
  2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
                   ` (3 preceding siblings ...)
  2006-06-27 15:42 ` tromey at gcc dot gnu dot org
@ 2006-06-27 15:46 ` tromey at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: tromey at gcc dot gnu dot org @ 2006-06-27 15:46 UTC (permalink / raw)
  To: java-prs



------- Comment #4 from tromey at gcc dot gnu dot org  2006-06-27 15:46 -------
I fixed this on the 4.1 branch and the trunk.
I'm not planning to backport to the 4.0 branch, but I would
approve it if someone wanted to.


-- 

tromey at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.1.2


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28178


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2006-06-27 15:46 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-06-27 14:03 [Bug libgcj/28178] New: jniEnv->DeleteLocalRef (null) fails hakan dot hjort at gmail dot com
2006-06-27 14:37 ` [Bug libgcj/28178] " hakan dot hjort at gmail dot com
2006-06-27 15:16 ` tromey at gcc dot gnu dot org
2006-06-27 15:33 ` tromey at gcc dot gnu dot org
2006-06-27 15:42 ` tromey at gcc dot gnu dot org
2006-06-27 15:46 ` tromey at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).