diff options
| author | Bruce Momjian <bruce@momjian.us> | 2002-08-27 03:56:35 +0000 |
|---|---|---|
| committer | Bruce Momjian <bruce@momjian.us> | 2002-08-27 03:56:35 +0000 |
| commit | dd912c697718e924387dd405413f94e481f8d0d4 (patch) | |
| tree | 120239674272c3c65400c56282c46a99b36d521a /src/backend/storage/ipc/shmqueue.c | |
| parent | e0a77f56e3e8f44b5014cb7584b33661b2e273e4 (diff) | |
| download | postgresql-dd912c697718e924387dd405413f94e481f8d0d4.tar.gz postgresql-dd912c697718e924387dd405413f94e481f8d0d4.zip | |
This patches replaces a few more usages of strcpy() and sprintf() when
copying into a fixed-size buffer (in this case, a buffer of
NAMEDATALEN bytes). AFAICT nothing to worry about here, but worth
fixing anyway...
Neil Conway
Diffstat (limited to 'src/backend/storage/ipc/shmqueue.c')
| -rw-r--r-- | src/backend/storage/ipc/shmqueue.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/backend/storage/ipc/shmqueue.c b/src/backend/storage/ipc/shmqueue.c index bbd98038ef1..726ae556c42 100644 --- a/src/backend/storage/ipc/shmqueue.c +++ b/src/backend/storage/ipc/shmqueue.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/storage/ipc/shmqueue.c,v 1.21 2002/06/20 20:29:35 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/storage/ipc/shmqueue.c,v 1.22 2002/08/27 03:56:35 momjian Exp $ * * NOTES * @@ -214,11 +214,11 @@ dumpQ(SHM_QUEUE *q, char *s) SHM_QUEUE *start = q; int count = 0; - sprintf(buf, "q prevs: %lx", MAKE_OFFSET(q)); + snprintf(buf, sizeof(buf), "q prevs: %lx", MAKE_OFFSET(q)); q = (SHM_QUEUE *) MAKE_PTR(q->prev); while (q != start) { - sprintf(elem, "--->%lx", MAKE_OFFSET(q)); + snprintf(elem, sizeof(elem), "--->%lx", MAKE_OFFSET(q)); strcat(buf, elem); q = (SHM_QUEUE *) MAKE_PTR(q->prev); if (q->prev == MAKE_OFFSET(q)) @@ -229,16 +229,16 @@ dumpQ(SHM_QUEUE *q, char *s) break; } } - sprintf(elem, "--->%lx", MAKE_OFFSET(q)); + snprintf(elem, sizeof(elem), "--->%lx", MAKE_OFFSET(q)); strcat(buf, elem); elog(SHMQUEUE_DEBUG_ELOG, "%s: %s", s, buf); - sprintf(buf, "q nexts: %lx", MAKE_OFFSET(q)); + snprintf(buf, sizeof(buf), "q nexts: %lx", MAKE_OFFSET(q)); count = 0; q = (SHM_QUEUE *) MAKE_PTR(q->next); while (q != start) { - sprintf(elem, "--->%lx", MAKE_OFFSET(q)); + snprintf(elem, sizeof(elem), "--->%lx", MAKE_OFFSET(q)); strcat(buf, elem); q = (SHM_QUEUE *) MAKE_PTR(q->next); if (q->next == MAKE_OFFSET(q)) @@ -249,7 +249,7 @@ dumpQ(SHM_QUEUE *q, char *s) break; } } - sprintf(elem, "--->%lx", MAKE_OFFSET(q)); + snprintf(elem, sizeof(elem), "--->%lx", MAKE_OFFSET(q)); strcat(buf, elem); elog(SHMQUEUE_DEBUG_ELOG, "%s: %s", s, buf); } |