Improve the MPD time parser

- reject negative values, which strtoul() happily accepts
 - deal with an arbitrary number of decimal digits
 - don't return milliseconds when we fail to parse seconds
This commit is contained in:
Přemysl Eric Janouch 2020-10-24 08:20:24 +02:00
parent 3934d9b1f9
commit c0119027b1
Signed by: p
GPG Key ID: A0420B94F92B9493
1 changed files with 18 additions and 9 deletions

View File

@ -3358,18 +3358,27 @@ mpd_read_time (const char *value, int *sec, int *optional_msec)
if (!value)
return;
char *end, *period = strchr (value, '.');
if (optional_msec && period)
{
unsigned long n = strtoul (period + 1, &end, 10);
if (*end)
char *end = NULL;
long n = strtol (value, &end, 10);
if (n < 0 || (*end && *end != '.'))
return;
// XXX: this relies on three decimal places
*optional_msec = MIN (INT_MAX, n);
int msec = 0;
if (*end == '.')
{
// In practice, MPD always uses three decimal digits
size_t digits = strspn (++end, "0123456789");
if (end[digits])
return;
if (digits--) msec += (*end++ - '0') * 100;
if (digits--) msec += (*end++ - '0') * 10;
if (digits--) msec += *end++ - '0';
}
unsigned long n = strtoul (value, &end, 10);
if (end == period || !*end)
*sec = MIN (INT_MAX, n);
if (optional_msec)
*optional_msec = msec;
}
static void