On Sun, Aug 7, 2016 at 12:16 AM, Christophe JAILLET <
[email protected]> 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.