summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDerrell Lipman <derrell.lipman@unwireduniverse.com>2008-03-06 09:00:37 -0500
committerDerrell Lipman <derrell.lipman@unwireduniverse.com>2008-03-06 09:00:37 -0500
commit5b80b9340fb64413f68d625cf5ca57344c0adbd7 (patch)
tree692ff8696df42c8a6b9839cffbaf861459016aef
parenta521f39c89e933aa11838e1189274c43c57ecfd6 (diff)
downloadsamba-5b80b9340fb64413f68d625cf5ca57344c0adbd7.tar.gz
samba-5b80b9340fb64413f68d625cf5ca57344c0adbd7.tar.bz2
samba-5b80b9340fb64413f68d625cf5ca57344c0adbd7.zip
Check for NULL pointers before dereferencing them.
(This used to be commit 6f65390cec218a6aac4370ee381f30439617dcec)
-rw-r--r--source3/libsmb/libsmb_server.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/source3/libsmb/libsmb_server.c b/source3/libsmb/libsmb_server.c
index 64eb1ea584..37612c6e39 100644
--- a/source3/libsmb/libsmb_server.c
+++ b/source3/libsmb/libsmb_server.c
@@ -144,6 +144,10 @@ SMBC_find_server(TALLOC_CTX *ctx,
SMBCSRV *srv;
int auth_called = 0;
+ if (!pp_workgroup || !pp_username || !pp_password) {
+ return NULL;
+ }
+
check_server_cache:
srv = smbc_getFunctionGetCachedServer(context)(context,
@@ -156,10 +160,6 @@ check_server_cache:
SMBC_call_auth_fn(ctx, context, server, share,
pp_workgroup, pp_username, pp_password);
- if (!pp_workgroup || !pp_username || !pp_password) {
- return NULL;
- }
-
/*
* However, smbc_auth_fn may have picked up info relating to
* an existing connection, so try for an existing connection