summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2003-02-15 21:41:01 +0000
committerAndrew Bartlett <abartlet@samba.org>2003-02-15 21:41:01 +0000
commit88a4d79e7b771ec8bcc903a7406e15cf4aeea248 (patch)
treece5502768d49f40d88ab44ba6f3ae350585f48f0
parent584206dccfd61ec49c729291b6f1874b4e38d2be (diff)
downloadsamba-88a4d79e7b771ec8bcc903a7406e15cf4aeea248.tar.gz
samba-88a4d79e7b771ec8bcc903a7406e15cf4aeea248.tar.bz2
samba-88a4d79e7b771ec8bcc903a7406e15cf4aeea248.zip
Don't return NULL pointers for now.
We should look into how to deal with NULL v "" strings, and the NTLMSSP code underneath properly at some stage. Andrew Bartlett (This used to be commit dc934412b0190ea75073cccddac45e74ebcd4a6b)
-rw-r--r--source3/libsmb/ntlmssp_parse.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/source3/libsmb/ntlmssp_parse.c b/source3/libsmb/ntlmssp_parse.c
index 6644a3db71..ac779a3906 100644
--- a/source3/libsmb/ntlmssp_parse.c
+++ b/source3/libsmb/ntlmssp_parse.c
@@ -234,9 +234,9 @@ BOOL msrpc_parse(const DATA_BLOB *blob,
pull_string(NULL, p, blob->data + ptr, sizeof(p),
len1,
STR_UNICODE|STR_NOALIGN);
- (*ps) = strdup(p);
+ (*ps) = smb_xstrdup(p);
} else {
- (*ps) = NULL;
+ (*ps) = smb_xstrdup("");
}
break;
case 'A':
@@ -255,9 +255,9 @@ BOOL msrpc_parse(const DATA_BLOB *blob,
pull_string(NULL, p, blob->data + ptr, sizeof(p),
len1,
STR_ASCII|STR_NOALIGN);
- (*ps) = strdup(p);
+ (*ps) = smb_xstrdup(p);
} else {
- (*ps) = NULL;
+ (*ps) = smb_xstrdup("");
}
break;
case 'B':