98aefe8772
'endptr' is appropriate internally in strtol(3) because it's a pointer to 'end', and 'end' itself is a pointer to one-after-the-last character of the numeric string. In other words, endptr == &end However, naming the pointer whose address we pass to strtol(3)'s 'endptr' feels wrong, and causes me trouble while parsing the code; I need to double check the number of dereferences, because something feels wrong in my head. Signed-off-by: Alejandro Colomar <alx@kernel.org>
77 lines
1.4 KiB
C
77 lines
1.4 KiB
C
/*
|
|
* SPDX-FileCopyrightText: 2008 , Nicolas François
|
|
*
|
|
* SPDX-License-Identifier: BSD-3-Clause
|
|
*/
|
|
|
|
|
|
#include <config.h>
|
|
|
|
#ident "$Id: $"
|
|
|
|
#include <ctype.h>
|
|
#include <stdlib.h>
|
|
|
|
#include "atoi/strtou_noneg.h"
|
|
#include "defines.h"
|
|
#include "prototypes.h"
|
|
|
|
|
|
/*
|
|
* Parse a range and indicate if the range is valid.
|
|
* Valid ranges are in the form:
|
|
* <long> -> min=max=long has_min has_max
|
|
* -<long> -> max=long !has_min has_max
|
|
* <long>- -> min=long has_min !has_max
|
|
* <long1>-<long2> -> min=long1 max=long2 has_min has_max
|
|
*/
|
|
int
|
|
getrange(const char *range,
|
|
unsigned long *min, bool *has_min,
|
|
unsigned long *max, bool *has_max)
|
|
{
|
|
char *end;
|
|
|
|
if (NULL == range)
|
|
return -1;
|
|
|
|
*has_min = false;
|
|
*has_max = false;
|
|
|
|
if ('-' == range[0]) {
|
|
end = range + 1;
|
|
goto parse_max;
|
|
}
|
|
|
|
errno = 0;
|
|
*min = strtoul_noneg(range, &end, 10);
|
|
if (end == range || 0 != errno)
|
|
return -1;
|
|
*has_min = true;
|
|
|
|
switch (*end++) {
|
|
case '\0':
|
|
*has_max = true;
|
|
*max = *min;
|
|
return 0; /* <long> */
|
|
|
|
case '-':
|
|
if ('\0' == *end)
|
|
return 0; /* <long>- */
|
|
parse_max:
|
|
if (!isdigit(*end))
|
|
return -1;
|
|
|
|
errno = 0;
|
|
*max = strtoul_noneg(end, &end, 10);
|
|
if ('\0' != *end || 0 != errno)
|
|
return -1;
|
|
*has_max = true;
|
|
|
|
return 0; /* <long>-<long>, or -<long> */
|
|
|
|
default:
|
|
return -1;
|
|
}
|
|
}
|