summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-04-15 13:49:59 +0200
committerKarolin Seeger <kseeger@samba.org>2008-04-21 14:19:15 +0200
commit32608d79e78eb899069ce4196e70b9f8613a531c (patch)
treee63036798d826af6f30ec6fc30a5e23f0b3909b6
parent832f54ead4b46b3574e6972651e0e3af07f962c7 (diff)
downloadsamba-32608d79e78eb899069ce4196e70b9f8613a531c.tar.gz
samba-32608d79e78eb899069ce4196e70b9f8613a531c.tar.xz
samba-32608d79e78eb899069ce4196e70b9f8613a531c.zip
libsmbconf: add handling of "NULL" strings to smbconf_add_string_to_array()
Michael (cherry picked from commit aba261a9143bc7ab681b1c57ccfc08da0cffcfe7)
-rw-r--r--source/lib/smbconf/smbconf_util.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/source/lib/smbconf/smbconf_util.c b/source/lib/smbconf/smbconf_util.c
index 1a3a0ded44b..20bd51a62bd 100644
--- a/source/lib/smbconf/smbconf_util.c
+++ b/source/lib/smbconf/smbconf_util.c
@@ -82,7 +82,7 @@ WERROR smbconf_add_string_to_array(TALLOC_CTX *mem_ctx,
{
char **new_array = NULL;
- if ((array == NULL) || (string == NULL)) {
+ if (array == NULL) {
return WERR_INVALID_PARAM;
}
@@ -91,10 +91,14 @@ WERROR smbconf_add_string_to_array(TALLOC_CTX *mem_ctx,
return WERR_NOMEM;
}
- new_array[count] = talloc_strdup(new_array, string);
- if (new_array[count] == NULL) {
- TALLOC_FREE(new_array);
- return WERR_NOMEM;
+ if (string == NULL) {
+ new_array[count] = NULL;
+ } else {
+ new_array[count] = talloc_strdup(new_array, string);
+ if (new_array[count] == NULL) {
+ TALLOC_FREE(new_array);
+ return WERR_NOMEM;
+ }
}
*array = new_array;