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