diff options
| author | Daniel Gustafsson <dgustafsson@postgresql.org> | 2025-04-30 23:00:47 +0200 |
|---|---|---|
| committer | Daniel Gustafsson <dgustafsson@postgresql.org> | 2025-04-30 23:00:47 +0200 |
| commit | 45e7e8ca9e9382e8f5e51fe0c085ea7f5d408ac8 (patch) | |
| tree | 516fc48958e8234ee9830d99d7a112f51431597f | |
| parent | 2d6745a66b12e980fb12bd06d583a66231b5ed5c (diff) | |
| download | postgresql-45e7e8ca9e9382e8f5e51fe0c085ea7f5d408ac8.tar.gz postgresql-45e7e8ca9e9382e8f5e51fe0c085ea7f5d408ac8.zip | |
Convert strncpy to strlcpy
We try to avoid using strncpy() due to the ease of which it can
be misused. Convert this callsite to use strlcpy() instead to
match similar codepaths in this file.
Suggested-by: Peter Eisentraut <peter@eisentraut.org>
Discussion: https://postgr.es/m/2a796830-de2d-4030-b480-d673f6cc5d94@eisentraut.org
| -rw-r--r-- | src/backend/utils/mmgr/mcxt.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/backend/utils/mmgr/mcxt.c b/src/backend/utils/mmgr/mcxt.c index 69884701fb4..7d28ca706eb 100644 --- a/src/backend/utils/mmgr/mcxt.c +++ b/src/backend/utils/mmgr/mcxt.c @@ -1664,8 +1664,7 @@ ProcessGetMemoryContextInterrupt(void) num_individual_stats = context_id + 1; meminfo[max_stats - 1].name = dsa_allocate(MemoryStatsDsaArea, namelen + 1); nameptr = dsa_get_address(MemoryStatsDsaArea, meminfo[max_stats - 1].name); - strncpy(nameptr, "Remaining Totals", namelen); - nameptr[namelen] = '\0'; + strlcpy(nameptr, "Remaining Totals", namelen + 1); meminfo[max_stats - 1].ident = InvalidDsaPointer; meminfo[max_stats - 1].path = InvalidDsaPointer; meminfo[max_stats - 1].type = 0; |