From 506c69a50e8d434b6b0c2c89b0402f220830644d Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Tue, 8 Jul 2014 00:59:48 +0200 Subject: [PATCH] BUILD: http: fix isdigit & isspace warnings on Solaris As usual, when touching any is* function, Solaris complains about the type of the element being checked. Better backport this to 1.5 since nobody knows what the emitted code looks like since macros are used instead of functions. --- src/proto_http.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/proto_http.c b/src/proto_http.c index 01fe62d092..4a862b054c 100644 --- a/src/proto_http.c +++ b/src/proto_http.c @@ -2143,22 +2143,22 @@ int parse_qvalue(const char *qvalue, const char **end) { int q = 1000; - if (!isdigit(*qvalue)) + if (!isdigit((unsigned char)*qvalue)) goto out; q = (*qvalue++ - '0') * 1000; if (*qvalue++ != '.') goto out; - if (!isdigit(*qvalue)) + if (!isdigit((unsigned char)*qvalue)) goto out; q += (*qvalue++ - '0') * 100; - if (!isdigit(*qvalue)) + if (!isdigit((unsigned char)*qvalue)) goto out; q += (*qvalue++ - '0') * 10; - if (!isdigit(*qvalue)) + if (!isdigit((unsigned char)*qvalue)) goto out; q += (*qvalue++ - '0') * 1; out: @@ -11226,7 +11226,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) while (1) { /* Jump spaces, quit if the end is detected. */ - while (al < end && isspace(*al)) + while (al < end && isspace((unsigned char)*al)) al++; if (al >= end) break; @@ -11235,7 +11235,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) token = al; /* Look for separator: isspace(), ',' or ';'. Next value if 0 length word. */ - while (al < end && *al != ';' && *al != ',' && !isspace(*al)) + while (al < end && *al != ';' && *al != ',' && !isspace((unsigned char)*al)) al++; if (al == token) goto expect_comma; @@ -11264,7 +11264,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) look_for_q: /* Jump spaces, quit if the end is detected. */ - while (al < end && isspace(*al)) + while (al < end && isspace((unsigned char)*al)) al++; if (al >= end) goto process_value; @@ -11283,7 +11283,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) al++; /* Jump spaces, process value if the end is detected. */ - while (al < end && isspace(*al)) + while (al < end && isspace((unsigned char)*al)) al++; if (al >= end) goto process_value; @@ -11294,7 +11294,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) al++; /* Jump spaces, process value if the end is detected. */ - while (al < end && isspace(*al)) + while (al < end && isspace((unsigned char)*al)) al++; if (al >= end) goto process_value; @@ -11305,7 +11305,7 @@ static int sample_conv_q_prefered(const struct arg *args, struct sample *smp) al++; /* Jump spaces, process value if the end is detected. */ - while (al < end && isspace(*al)) + while (al < end && isspace((unsigned char)*al)) al++; if (al >= end) goto process_value;