public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
* [SCM]  master: Fix off-by-one bounds check.
@ 2008-06-03 22:25 cagney
  0 siblings, 0 replies; only message in thread
From: cagney @ 2008-06-03 22:25 UTC (permalink / raw)
  To: frysk-cvs

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


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2008-06-03 22:25 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-06-03 22:25 [SCM] master: Fix off-by-one bounds check cagney

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).