You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@subversion.apache.org by hw...@apache.org on 2010/05/21 15:48:37 UTC

svn commit: r947006 - /subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp

Author: hwright
Date: Fri May 21 13:48:36 2010
New Revision: 947006

URL: http://svn.apache.org/viewvc?rev=947006&view=rev
Log:
* subversion/bindings/javahl/native/Outputer.cpp
  (write): Add a call to DeleteLocalRef() to prevent a leaking local
    reference.

Patch by: Byeongcheol Lee <li...@gmail.com>
Tweaked by: me

Modified:
    subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp

Modified: subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp?rev=947006&r1=947005&r2=947006&view=diff
==============================================================================
--- subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp (original)
+++ subversion/trunk/subversion/bindings/javahl/native/Outputer.cpp Fri May 21 13:48:36 2010
@@ -106,6 +106,10 @@ svn_error_t *Outputer::write(void *baton
   if (JNIUtil::isJavaExceptionThrown())
     return SVN_NO_ERROR;
 
+  env->DeleteLocalRef(data);
+  if (JNIUtil::isJavaExceptionThrown())
+    return SVN_NO_ERROR;
+
   // return the number of bytes written
   *len = written;