diff options
author | Julien Cristau <jcristau@debian.org> | 2016-03-02 21:22:25 +0100 |
---|---|---|
committer | Julien Cristau <jcristau@debian.org> | 2016-03-02 21:30:17 +0100 |
commit | 8534d18bf6af7446b0759b39089323cf7c51c1d5 (patch) | |
tree | 19efbb83e07915d3287449a0ccff7b85e2f8020d | |
parent | 2aef1f08de4e0f74436bc94301ff0fbabc1ea57b (diff) |
xwin: no need to free auth data if AddResource fails
This is taken care of by SecurityDeleteAuthorization
Signed-off-by: Julien Cristau <jcristau@debian.org>
-rw-r--r-- | hw/xwin/winauth.c | 17 |
1 files changed, 3 insertions, 14 deletions
diff --git a/hw/xwin/winauth.c b/hw/xwin/winauth.c index 7efa1c00f..7be7dcae1 100644 --- a/hw/xwin/winauth.c +++ b/hw/xwin/winauth.c @@ -114,7 +114,6 @@ GenerateAuthorization(unsigned name_length, Bool winGenerateAuthorization(void) { - Bool fFreeAuth = FALSE; SecurityAuthorizationPtr pAuth = NULL; /* Call OS layer to generate authorization key */ @@ -123,7 +122,7 @@ winGenerateAuthorization(void) 0, NULL, &g_uiAuthDataLen, &g_pAuthData); if ((XID) ~0L == g_authId) { ErrorF("winGenerateAuthorization - GenerateAuthorization failed\n"); - goto auth_bailout; + return FALSE; } else { @@ -139,7 +138,7 @@ winGenerateAuthorization(void) if (!(pAuth)) { ErrorF("winGenerateAuthorization - Failed allocating " "SecurityAuthorizationPtr.\n"); - goto auth_bailout; + return FALSE; } /* Fill in the auth fields */ @@ -155,21 +154,11 @@ winGenerateAuthorization(void) /* Add the authorization to the server's auth list */ if (!AddResource(g_authId, SecurityAuthorizationResType, pAuth)) { ErrorF("winGenerateAuthorization - AddResource failed for auth.\n"); - fFreeAuth = TRUE; - goto auth_bailout; + return FALSE; } - - /* Don't free the auth data, since it is still used internally */ - pAuth = NULL; #endif return TRUE; - - auth_bailout: - if (fFreeAuth) - free(pAuth); - - return FALSE; } /* Use our generated cookie for authentication */ |