diff options
| author | Michael Paquier | 2025-12-16 23:58:58 +0000 |
|---|---|---|
| committer | Michael Paquier | 2025-12-16 23:58:58 +0000 |
| commit | 1d325ad99cb2dec0e8b45ba36909ee0a497d2a57 (patch) | |
| tree | c3976e5e513d9c55f890578ac42ab8ecb2600e36 | |
| parent | 0a90df58cf38cf68d59c6841513be98aeeff250e (diff) | |
Remove useless code in InjectionPointAttach()
strlcpy() ensures that a target string is zero-terminated, so there is
no need to enforce it a second time in this code. This simplification
could have been done in 0eb23285a257.
Author: Feilong Meng <feelingmeng@foxmail.com>
Reviewed-by: Chao Li <li.evan.chao@gmail.com>
Discussion: https://postgr.es/m/tencent_771178777C5BC17FCB7F7A1771CD1FFD5708@qq.com
| -rw-r--r-- | src/backend/utils/misc/injection_point.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/backend/utils/misc/injection_point.c b/src/backend/utils/misc/injection_point.c index 54a9fe8e163..4945da458b1 100644 --- a/src/backend/utils/misc/injection_point.c +++ b/src/backend/utils/misc/injection_point.c @@ -331,11 +331,8 @@ InjectionPointAttach(const char *name, /* Save the entry */ strlcpy(entry->name, name, sizeof(entry->name)); - entry->name[INJ_NAME_MAXLEN - 1] = '\0'; strlcpy(entry->library, library, sizeof(entry->library)); - entry->library[INJ_LIB_MAXLEN - 1] = '\0'; strlcpy(entry->function, function, sizeof(entry->function)); - entry->function[INJ_FUNC_MAXLEN - 1] = '\0'; if (private_data != NULL) memcpy(entry->private_data, private_data, private_data_size); |