On Sun, Aug 7, 2016 at 12:16 AM, Christophe JAILLET < christophe.jail...@wanadoo.fr> wrote:
> > Anyway, while looking at it, I noticed the code below. > I think that it is a leftover because I don't see any reason to handle > these 2 string comparisons differently. > >> Agreed, this looks like a simple oversight, feel free to simply fix trunk... > @@ -542,7 +542,7 @@ static const char *set_qop(cmd_parms *cm >> if (!strcasecmp(op, "auth-int")) { >> > strcasecmp unchanged here. > > return "AuthDigestQop auth-int is not implemented"; >> } >> - else if (strcasecmp(op, "auth")) { >> + else if (ap_casecmpstr(op, "auth")) { >> > but updated there.