Hi,

I checked all the previous string handling errors and most of them were 
already fixed by You. However there were a few left and attached patch 
should fix the rest of them.

I used StringInfo only in 2 places and both of them are inside debug 
ifdefs. Only performance penalty will come from using strlen() like all 
the other code does.

I also modified some of the already patched parts by changing 
snprintf(buf, 2 * BUFSIZE, ... style lines to
snprintf(buf, sizeof(buf), ... where buf is an array.

This patch also passes all regression testing:
======================
  All 89 tests passed.
======================

Patch is in -c format as requested and also available from 
http://suihkari.baana.suomi.net/postgresql/patches/postgresql-CVS-2002-08-29-sprintf.patch

- Jukka
Fri, 30 Aug 2002 00:12:22 +0300
Jukka Holappa <jukkaho@mail.student.oulu.fi>
[PATCH] Sprintf() patch against current CVS tree.