| 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: | Whole Thread | Raw Message | 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 | 
| 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 | 
| 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 |