diff options
author | Andrew Bartlett <abartlet@samba.org> | 2003-02-15 21:41:01 +0000 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2003-02-15 21:41:01 +0000 |
commit | 88a4d79e7b771ec8bcc903a7406e15cf4aeea248 (patch) | |
tree | ce5502768d49f40d88ab44ba6f3ae350585f48f0 | |
parent | 584206dccfd61ec49c729291b6f1874b4e38d2be (diff) | |
download | samba-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.c | 8 |
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': |