diff options
| author | Max Kellermann <max@duempel.org> | 2008-08-29 09:38:58 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2008-08-29 09:38:58 +0200 |
| commit | 43c389b961c609f9c705cfe14ed429082ac9115a (patch) | |
| tree | 61e93ef93115a5f96c5981cae1fd7a36e789d9f7 /src/path.c | |
| parent | 92b757674ebcf5cf90e8adb66e7583edf1bc604e (diff) | |
| download | mpd-43c389b961c609f9c705cfe14ed429082ac9115a.tar.gz mpd-43c389b961c609f9c705cfe14ed429082ac9115a.tar.xz mpd-43c389b961c609f9c705cfe14ed429082ac9115a.zip | |
added "length" parameter to validUtf8String()
At several places, we create temporary copies of non-null-terminated
strings, just to use them in functions like validUtf8String(). We can
save this temporary allocation and avoid heap fragmentation if we
add a length parameter instead of expecting a null-terminated string.
Diffstat (limited to '')
| -rw-r--r-- | src/path.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/path.c b/src/path.c index 6aaff84cf..ceb00c5de 100644 --- a/src/path.c +++ b/src/path.c @@ -47,7 +47,7 @@ static char *path_conv_charset(char *dest, const char *to, char *fs_charset_to_utf8(char *dst, const char *str) { char *ret = path_conv_charset(dst, "UTF-8", fsCharset, str); - return (ret && !validUtf8String(ret)) ? NULL : ret; + return (ret && !validUtf8String(ret, strlen(ret))) ? NULL : ret; } char *utf8_to_fs_charset(char *dst, const char *str) |