os: xdmcp: simplify AddAuth handler call

No need for loading the address into temporary variable and later doing
indirect call, since we can easily do direct call in each branch.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This commit is contained in:
Enrico Weigelt, metux IT consult 2025-03-10 15:02:45 +01:00
parent 6c6e0b09a1
commit 1629400be5

View File

@ -902,13 +902,14 @@ XdmcpCheckAuthentication(ARRAY8Ptr Name, ARRAY8Ptr Data, int packet_type)
static int static int
XdmcpAddAuthorization(ARRAY8Ptr name, ARRAY8Ptr data) XdmcpAddAuthorization(ARRAY8Ptr name, ARRAY8Ptr data)
{ {
AddAuthorFunc AddAuth;
if (AuthenticationFuncs && AuthenticationFuncs->AddAuth) if (AuthenticationFuncs && AuthenticationFuncs->AddAuth)
AddAuth = AuthenticationFuncs->AddAuth; return AuthenticationFuncs->AddAuth(
(unsigned short) name->length,
(char *) name->data,
(unsigned short) data->length, (char *) data->data);
else else
AddAuth = AddAuthorization; return AddAuthorization(
return (*AddAuth) ((unsigned short) name->length, (unsigned short) name->length,
(char *) name->data, (char *) name->data,
(unsigned short) data->length, (char *) data->data); (unsigned short) data->length, (char *) data->data);
} }