From 702dd3da6297967ce140771110f85d61e0dff1e5 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Fri, 24 Jun 2005 14:25:18 +0000 Subject: r7877: Attempt to fix a smb_panic reported by Pavel Rochnyack. Volker (This used to be commit 7d1b890fead61551465e2a972e4097d9c1a4d6fd) --- source3/nsswitch/winbindd_user.c | 23 +++++++++++++++-------- 1 file changed, 15 insertions(+), 8 deletions(-) (limited to 'source3/nsswitch/winbindd_user.c') diff --git a/source3/nsswitch/winbindd_user.c b/source3/nsswitch/winbindd_user.c index 55b63a8182..82ab79eb4c 100644 --- a/source3/nsswitch/winbindd_user.c +++ b/source3/nsswitch/winbindd_user.c @@ -401,7 +401,7 @@ void winbindd_getpwuid(struct winbindd_cli_state *state) /* Rewind file pointer for ntdom passwd database */ -void winbindd_setpwent(struct winbindd_cli_state *state) +static BOOL winbindd_setpwent_internal(struct winbindd_cli_state *state) { struct winbindd_domain *domain; @@ -410,8 +410,7 @@ void winbindd_setpwent(struct winbindd_cli_state *state) /* Check user has enabled this */ if (!lp_winbind_enum_users()) { - request_error(state); - return; + return False; } /* Free old static data if it exists */ @@ -425,7 +424,7 @@ void winbindd_setpwent(struct winbindd_cli_state *state) /* add any local users we have */ if ( (domain_state = (struct getent_state *)malloc(sizeof(struct getent_state))) == NULL ) - return WINBINDD_ERROR; + return False; ZERO_STRUCTP(domain_state); @@ -453,8 +452,7 @@ void winbindd_setpwent(struct winbindd_cli_state *state) if ((domain_state = SMB_MALLOC_P(struct getent_state)) == NULL) { DEBUG(0, ("malloc failed\n")); - request_error(state); - return; + return False; } ZERO_STRUCTP(domain_state); @@ -467,7 +465,16 @@ void winbindd_setpwent(struct winbindd_cli_state *state) } state->getpwent_initialized = True; - request_ok(state); + return True; +} + +void winbindd_setpwent(struct winbindd_cli_state *state) +{ + if (winbindd_setpwent_internal(state)) { + request_ok(state); + } else { + request_error(state); + } } /* Close file pointer to ntdom passwd database */ @@ -603,7 +610,7 @@ void winbindd_getpwent(struct winbindd_cli_state *state) user_list = (struct winbindd_pw *)state->response.extra_data; if (!state->getpwent_initialized) - winbindd_setpwent(state); + winbindd_setpwent_internal(state); if (!(ent = state->getpwent_state)) { request_error(state); -- cgit