diff options
| author | Jeremy Allison <jra@samba.org> | 2011-07-12 13:06:13 -0700 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2011-07-26 21:53:35 +0200 |
| commit | d4dc24d484203753650a6c1fd55ff92e4bafc9a4 (patch) | |
| tree | 6ec8a72170de72461574eec0e8058ebc81669242 | |
| parent | 1deaa86ad7c2d1eef179cfdc73703ff51416d394 (diff) | |
| download | samba-d4dc24d484203753650a6c1fd55ff92e4bafc9a4.tar.gz samba-d4dc24d484203753650a6c1fd55ff92e4bafc9a4.tar.xz samba-d4dc24d484203753650a6c1fd55ff92e4bafc9a4.zip | |
Fix bug #Bug 8304 - Uninitialized variable referenced in error path.
princ must not be used uninitialized. auth_data is not used at all.
(cherry picked from commit ba6f88a6720358bea75b162f193182b724b9411c)
| -rw-r--r-- | source3/rpc_server/dcesrv_gssapi.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/source3/rpc_server/dcesrv_gssapi.c b/source3/rpc_server/dcesrv_gssapi.c index 25d85a6730c..f027961b309 100644 --- a/source3/rpc_server/dcesrv_gssapi.c +++ b/source3/rpc_server/dcesrv_gssapi.c @@ -105,7 +105,6 @@ NTSTATUS gssapi_server_get_user_info(struct gse_context *gse_ctx, struct auth_serversupplied_info **server_info) { TALLOC_CTX *tmp_ctx; - DATA_BLOB auth_data; DATA_BLOB pac; struct PAC_DATA *pac_data; struct PAC_LOGON_INFO *logon_info = NULL; @@ -135,6 +134,11 @@ NTSTATUS gssapi_server_get_user_info(struct gse_context *gse_ctx, goto done; } + status = gse_get_client_name(gse_ctx, tmp_ctx, &princ_name); + if (!NT_STATUS_IS_OK(status)) { + goto done; + } + pac_data = talloc_zero(tmp_ctx, struct PAC_DATA); if (!pac_data) { status = NT_STATUS_NO_MEMORY; |