This is the mail archive of the frysk-cvs@sources.redhat.com mailing list for the frysk project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[SCM] master: Fix off-by-one bounds check.


The branch, master has been updated
       via  d64a49dffa0bbf7e2e558f09c602b80b14f42063 (commit)
      from  b4b0e9bf226bc91db7cf802735d8791630504d0c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit d64a49dffa0bbf7e2e558f09c602b80b14f42063
Author: Andrew Cagney <cagney@redhat.com>
Date:   Tue Jun 3 18:24:03 2008 -0400

    Fix off-by-one bounds check.
    
    frysk-sys/jnixx/ChangeLog
    2008-06-03  Andrew Cagney  <cagney@redhat.com>
    
    	* bounds.hxx (verifyBounds): Allow bound to reach array end.

-----------------------------------------------------------------------

Summary of changes:
 frysk-sys/jnixx/ChangeLog  |    4 ++++
 frysk-sys/jnixx/bounds.hxx |    3 ---
 2 files changed, 4 insertions(+), 3 deletions(-)

First 500 lines of diff:
diff --git a/frysk-sys/jnixx/ChangeLog b/frysk-sys/jnixx/ChangeLog
index 4cc0ec7..593f456 100644
--- a/frysk-sys/jnixx/ChangeLog
+++ b/frysk-sys/jnixx/ChangeLog
@@ -1,3 +1,7 @@
+2008-06-03  Andrew Cagney  <cagney@redhat.com>
+
+	* bounds.hxx (verifyBounds): Allow bound to reach array end.
+
 2008-05-25  Andrew Cagney  <cagney@redhat.com>
 
 	* elements.hxx (jbyteArrayElements): Rename ArrayBytes.
diff --git a/frysk-sys/jnixx/bounds.hxx b/frysk-sys/jnixx/bounds.hxx
index f79a043..01aa4b7 100644
--- a/frysk-sys/jnixx/bounds.hxx
+++ b/frysk-sys/jnixx/bounds.hxx
@@ -57,9 +57,6 @@ inline void verifyBounds(::jnixx::env env, ::jnixx::jbyteArray data,
   if (start < 0) {
     ::java::lang::ArrayIndexOutOfBoundsException::New(env, start).Throw(env);
   }
-  if (start >= data.GetArrayLength(env)) {
-    ::java::lang::ArrayIndexOutOfBoundsException::New(env, start).Throw(env);
-  }
   if (length < 0) {
     ::java::lang::ArrayIndexOutOfBoundsException::New(env, length).Throw(env);
   }


hooks/post-receive
--
frysk system monitor/debugger


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]