From b70f23c2b581c5d455362ab37f4846de9a910055 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Wed, 8 Aug 2012 17:01:00 -0700 Subject: Correctly check for errors in strlower_m() returns. --- source3/rpc_server/dssetup/srv_dssetup_nt.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'source3/rpc_server/dssetup') diff --git a/source3/rpc_server/dssetup/srv_dssetup_nt.c b/source3/rpc_server/dssetup/srv_dssetup_nt.c index fef2dbd7e0..1a3998b8be 100644 --- a/source3/rpc_server/dssetup/srv_dssetup_nt.c +++ b/source3/rpc_server/dssetup/srv_dssetup_nt.c @@ -78,7 +78,9 @@ static WERROR fill_dsrole_dominfo_basic(TALLOC_CTX *ctx, if (!dnsdomain) { return WERR_NOMEM; } - strlower_m(dnsdomain); + if (!strlower_m(dnsdomain)) { + return WERR_INVALID_PARAM; + } basic->dns_domain = dnsdomain; /* FIXME!! We really should fill in the correct forest -- cgit