pg.ddx.io  pgsql-interfaces@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Gunnar R|nning <gunnar@candleweb.no>
To: Richard Bullington-McGuire <rbulling@microstate.com>
Cc: pgsql-patches@postgresql.org
Subject: Re: Fixes and enhancements to JDBC driver (take 2)
Date: 18 Jan 2001 14:49:23 +0100
Message-ID: <x6bst5dl7g.fsf@thor.candleweb.no> (raw)
References: <Pine.LNX.4.10.10101171425190.26916-200000@polymorphic.microstate.com>

Richard Bullington-McGuire <rbulling@microstate.com> writes:

> * A serious off-by-one error existed in both of the BytePoolDim classes in
> the org.postgresql.PG_Stream class. On a request for a byte pool for the
> maximum size, the old code would generate an
> ArrayIndexOutOfBoundsException. I fixed this by increasing the size of
> the array by one more than the maximum buffer size, and altering the
> supporting methods to initialize and clean up the expanded array.

This BytePoolDim code got other problems as well. I posted another
implementation of the same pooling idea back in October, but it never got
in to replace the current buggy code. As I wrote in pgsql-general earlier
today, I will look into creating a new patch with another pooling
implementation that should work a lot better. 

regards, 

	Gunnar



view thread (8+ messages)  latest in thread

Message-ID: <x6bst5dl7g.fsf@thor.candleweb.no>
Permalink:  ../x6bst5dl7g.fsf@thor.candleweb.no/
Also on:    postgresql.org/message-id/x6bst5dl7g.fsf@thor.candleweb.no

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-interfaces@postgresql.org
  Cc: gunnar@candleweb.no, rbulling@microstate.com, pgsql-patches@postgresql.org
  Subject: Re: Fixes and enhancements to JDBC driver (take 2)
  In-Reply-To: <x6bst5dl7g.fsf@thor.candleweb.no>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox