From 526e875cec15761099438e17df3f56bc2bd5b761 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 8 Aug 2012 15:35:28 -0700 Subject: Check error returns from strupper_m() (in all reasonable places). --- libgpo/gpext/gpext.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'libgpo') diff --git a/libgpo/gpext/gpext.c b/libgpo/gpext/gpext.c index 8bda729d69..df84475754 100644 --- a/libgpo/gpext/gpext.c +++ b/libgpo/gpext/gpext.c @@ -440,7 +440,9 @@ static WERROR gp_extension_store_reg_entry(TALLOC_CTX *mem_ctx, subkeyname = GUID_string2(mem_ctx, &entry->guid); W_ERROR_HAVE_NO_MEMORY(subkeyname); - strupper_m(discard_const_p(char, subkeyname)); + if (!strupper_m(discard_const_p(char, subkeyname))) { + return WERR_INVALID_PARAM; + } werr = gp_store_reg_subkey(mem_ctx, subkeyname, -- cgit