Patch for handling long null terminated strings in JDBC driver

From: Barry Lind <barry(at)xythos(dot)com>
To: "pgsql-patches(at)postgresql(dot)org" <pgsql-patches(at)postgresql(dot)org>
Cc: pgsql-jdbc(at)postgresql(dot)org, swampler(at)noao(dot)edu
Subject: Patch for handling long null terminated strings in JDBC driver
Date: 2001-07-12 21:16:41
Message-ID: 3B4E13B9.30205@xythos.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc pgsql-patches

The attached patch fixes problems with the JDBC driver handling long
null terminated strings. The FE/BE protocol sends in some cases null
terminated strings to the client. The docs for the FE/BE protocol state
that there is no limit on the size of a null terminated string sent to
the client and a client should be coded using an expanding buffer to
deal with large strings. The old code did not do this and gave an error
if a null terminated string was greater than either 4 or 8K. It appears
that with the advent of TOAST very long SQL statements are becoming more
common, and apparently some error messages from the backend include the
SQL statement thus easily exceeding the 8K limit in the old code.

In fixing I also cleaned up some calls in the JDBC fastpath code that
were not doing character set conversion under multibyte, and removed
some methods that were no longer needed. I also removed a potential
threading problem with a shared variable that was being used in
Connection.java.

Thanks to Steve Wampler for discovering the problem and sending the
initial diffs that were the basis of this patch.

thanks,
--Barry

Attachment Content-Type Size
patch.txt text/plain 11.6 KB

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Dejan Vucinic 2001-07-12 21:22:45 Resolved: can't write a BLOB, 7.1.2
Previous Message Bruce Momjian 2001-07-12 20:34:59 Re: Re: [PATCHES] Re: [PATCH] Cleanup of JDBC character encoding

Browse pgsql-patches by date

  From Date Subject
Next Message Martijn van Oosterhout 2001-07-12 23:28:44 Patch to add support for partial indices
Previous Message Bruce Momjian 2001-07-12 20:35:51 Re: patch for spelling mistake