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 |
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 |