denyMessage() vs setDenyMessage()

[Alex Rousskov]

Hello,

    Should the following calls use setDenyMessage() instead?

> ./auth/negotiate/UserRequest.cc:            
> auth_user_request->denyMessage("Authentication in progress");
> ./auth/negotiate/UserRequest.cc:            
> auth_user_request->denyMessage("NTLM authentication requires a persistent 
> connection");
> ./auth/negotiate/UserRequest.cc:        auth_user_request->denyMessage("Login 
> successful");
> ./auth/negotiate/UserRequest.cc:        auth_user_request->denyMessage(arg);
> ./auth/negotiate/UserRequest.cc:        auth_user_request->denyMessage(blob);
> ./auth/ntlm/UserRequest.cc:            
> auth_user_request->denyMessage("Authentication in progress");
> ./auth/ntlm/UserRequest.cc:            auth_user_request->denyMessage("NTLM 
> authentication requires a persistent connection");
> ./auth/ntlm/UserRequest.cc:        auth_user_request->denyMessage("Login 
> successful");
> ./auth/ntlm/UserRequest.cc:        auth_user_request->denyMessage(blob);
> ./auth/ntlm/UserRequest.cc:        auth_user_request->denyMessage(blob);


Thank you,

Alex.

> char const *
> Auth::UserRequest::denyMessage(char const * const default_message)
> {   
>     if (this == NULL || getDenyMessage() == NULL) {
>         return default_message;
>     }
> 
>     return getDenyMessage();
> }