[PATCH] Sprintf() patch against current CVS tree.

From: Jukka Holappa <jukkaho(at)mail(dot)student(dot)oulu(dot)fi>
To: pgsql-hackers(at)postgresql(dot)org
Subject: [PATCH] Sprintf() patch against current CVS tree.
Date: 2002-08-29 21:12:22
Message-ID: 3D6E8E36.8010604@mail.student.oulu.fi
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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

Attachment Content-Type Size
postgresql-CVS-2002-08-29-sprintf.patch text/plain 45.4 KB

Browse pgsql-hackers by date

  From Date Subject
Next Message Jukka Holappa 2002-08-29 21:29:03 [PATCH] Sprintf() patch against current CVS tree.
Previous Message Bruce Momjian 2002-08-29 19:55:27 Re: README.tsearch.diff for CVS