X-Original-To: pgsql-bugs-postgresql.org@localhost.postgresql.org Received: from localhost (unknown [200.46.204.2]) by svr1.postgresql.org (Postfix) with ESMTP id 9D13BD1B57A for ; Thu, 30 Oct 2003 21:14:09 +0000 (GMT) Received: from svr1.postgresql.org ([200.46.204.71]) by localhost (neptune.hub.org [200.46.204.2]) (amavisd-new, port 10024) with ESMTP id 41444-03 for ; Thu, 30 Oct 2003 17:13:39 -0400 (AST) Received: from web40607.mail.yahoo.com (web40607.mail.yahoo.com [66.218.78.144]) by svr1.postgresql.org (Postfix) with SMTP id 69F92D1B52D for ; Thu, 30 Oct 2003 17:13:37 -0400 (AST) Message-ID: <20031030211338.32691.qmail@web40607.mail.yahoo.com> Received: from [135.214.154.104] by web40607.mail.yahoo.com via HTTP; Thu, 30 Oct 2003 13:13:38 PST Date: Thu, 30 Oct 2003 13:13:38 -0800 (PST) From: Brett Schwarz Subject: Re: [INTERFACES] pgtcl large object read/write corrupts binary data To: Tom Lane , ljb Cc: pgsql-bugs@postgresql.org, pgsql-interfaces@postgresql.org In-Reply-To: <15956.1067452389@sss.pgh.pa.us> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Virus-Scanned: by amavisd-new at postgresql.org X-Archive-Number: 200310/193 X-Sequence-Number: 7042 --- Tom Lane wrote: > ljb writes: > > [Using PostgreSQL-7.3.4 and -7.4beta5, Tcl-8.4.x.] > > Binary data written to a Large Object with > libpgtcl's pg_lo_write is > > corrupted. Tcl is mangling the data - something > to do with UTF-8 > > conversion. Example: 0x80 becomes 0xc2 0x80, and > 0xff becomes 0xc3 0xbf. > > ... > > Proposed Patch: (I think this requires Tcl >= > 8.1) > > Yeah, it appears that ByteArrayObj was added in Tcl > 8.1. We were > already requiring Tcl 8.0 of course. Does anyone > think that it's > important to continue support for Tcl 8.0.* in > libpgtcl? > I don't think it is important to require Tcl 8.0 anymore. Tcl 8.0 came out in 1998. We might be able to get rid of some of the version checks as well in the code. --brett __________________________________ Do you Yahoo!? Exclusive Video Premiere - Britney Spears http://launch.yahoo.com/promos/britneyspears/