Skip to content

Commit 5c6339b

Browse files
committed
Improve DynamicMeter_getUiName() string length logic
Use 'size_t' for internal 'len' variables and use String_safeStrncpy() for copying strings. Using 'size_t' for string length prevents a "-Wshorten-64-to-32" warning which is enabled by default in Clang 19.
1 parent 7a0753c commit 5c6339b

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

DynamicMeter.c

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,13 @@ static void DynamicMeter_getUiName(const Meter* this, char* name, size_t length)
105105
if (meter) {
106106
const char* uiName = meter->caption;
107107
if (uiName) {
108-
int len = strlen(uiName);
108+
size_t len = strlen(uiName);
109109
if (len > 2 && uiName[len - 2] == ':')
110110
len -= 2;
111-
xSnprintf(name, length, "%.*s", len, uiName);
111+
112+
String_safeStrncpy(name, uiName, MINIMUM(length, len + 1));
112113
} else {
113-
xSnprintf(name, length, "%s", meter->name);
114+
String_safeStrncpy(name, meter->name, length);
114115
}
115116
}
116117
}

0 commit comments

Comments
 (0)