summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorC. Davis <cd.rattan@gmail.com>2012-08-08 14:37:20 -0700
committerMichael Adam <obnox@samba.org>2013-04-29 13:06:10 +0200
commit5cfbf7316c16c0205ff87f7980e05a2ab1671709 (patch)
treebff8eccb58de29bb61910d3b45f04495ec97d6b7 /source3
parent360f31d52b30ef14e8a584f62ff61cc55c95ed53 (diff)
downloadsamba-5cfbf7316c16c0205ff87f7980e05a2ab1671709.tar.gz
samba-5cfbf7316c16c0205ff87f7980e05a2ab1671709.tar.bz2
samba-5cfbf7316c16c0205ff87f7980e05a2ab1671709.zip
regedit: Don't pass empty value names to new_item().
Reviewed-by: Andreas Schneider <asn@samba.org> Reviewed-by: Michael Adam <obnox@samba.org>
Diffstat (limited to 'source3')
-rw-r--r--source3/utils/regedit_valuelist.c15
1 files changed, 13 insertions, 2 deletions
diff --git a/source3/utils/regedit_valuelist.c b/source3/utils/regedit_valuelist.c
index c09b22107f..644375283d 100644
--- a/source3/utils/regedit_valuelist.c
+++ b/source3/utils/regedit_valuelist.c
@@ -194,6 +194,8 @@ WERROR value_list_load(struct value_list *vl, struct registry_key *key)
struct value_item *vitem;
ITEM **new_items;
WERROR rv;
+ static const char *empty_name = "(empty)";
+ const char *name;
unpost_menu(vl->menu);
@@ -237,8 +239,17 @@ WERROR value_list_load(struct value_list *vl, struct registry_key *key)
return rv;
}
- new_items[idx] = new_item(vitem->value_name,
- vitem->value_desc);
+ /* ncurses won't accept empty strings in menu items */
+ name = vitem->value_name;
+ if (name[0] == '\0') {
+ name = empty_name;
+ }
+ new_items[idx] = new_item(name, vitem->value_desc);
+ if (new_items[idx] == NULL) {
+ talloc_free(vitem);
+ return WERR_NOMEM;
+ }
+
set_item_userptr(new_items[idx], vitem);
}