diff options
author | Günther Deschner <gd@samba.org> | 2007-06-21 14:04:55 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:23:30 -0500 |
commit | f4eb916e856395bf1055c5b3f0583f816cd5c2b5 (patch) | |
tree | d642ffbec4f5cd3474a3f83a5bb891699ca35e54 /source3/nsswitch/winbindd_cm.c | |
parent | fd881f40670792452247de5f6711b5a943612f03 (diff) | |
download | samba-f4eb916e856395bf1055c5b3f0583f816cd5c2b5.tar.gz samba-f4eb916e856395bf1055c5b3f0583f816cd5c2b5.tar.bz2 samba-f4eb916e856395bf1055c5b3f0583f816cd5c2b5.zip |
r23565: Avoid double NULL pointer checks.
Guenther
(This used to be commit 5456ea59ba12593b0aac9745b41cdd0f5ec0a559)
Diffstat (limited to 'source3/nsswitch/winbindd_cm.c')
-rw-r--r-- | source3/nsswitch/winbindd_cm.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/source3/nsswitch/winbindd_cm.c b/source3/nsswitch/winbindd_cm.c index c5e014192e..0571cdad60 100644 --- a/source3/nsswitch/winbindd_cm.c +++ b/source3/nsswitch/winbindd_cm.c @@ -258,9 +258,7 @@ static void check_domain_online_handler(struct event_context *ctx, "%s (online = %s)\n", domain->name, domain->online ? "True" : "False" )); - if (domain->check_online_event) { - TALLOC_FREE(domain->check_online_event); - } + TALLOC_FREE(domain->check_online_event); /* Are we still in "startup" mode ? */ @@ -312,9 +310,7 @@ void set_domain_offline(struct winbindd_domain *domain) DEBUG(10,("set_domain_offline: called for domain %s\n", domain->name )); - if (domain->check_online_event) { - TALLOC_FREE(domain->check_online_event); - } + TALLOC_FREE(domain->check_online_event); if (domain->internal) { DEBUG(3,("set_domain_offline: domain %s is internal - logic error.\n", @@ -426,9 +422,7 @@ static void set_domain_online(struct winbindd_domain *domain) /* Ensure we have no online timeout checks. */ domain->check_online_timeout = 0; - if (domain->check_online_event) { - TALLOC_FREE(domain->check_online_event); - } + TALLOC_FREE(domain->check_online_event); /* Ensure we ignore any pending child messages. */ messaging_deregister(winbind_messaging_context(), |