diff options
| author | Andreas Schneider <asn@samba.org> | 2012-12-10 17:36:39 +0100 |
|---|---|---|
| committer | Günther Deschner <gd@samba.org> | 2012-12-12 09:42:32 +0100 |
| commit | e4e3293b482d48bc6a503e1280fe2a20556548ef (patch) | |
| tree | 22153f1c75487e36d46379f3f14abeedbcdc6479 /source3/utils | |
| parent | f70c56b74769711e358860655bbb5c2329ad005b (diff) | |
| download | samba-e4e3293b482d48bc6a503e1280fe2a20556548ef.tar.gz samba-e4e3293b482d48bc6a503e1280fe2a20556548ef.tar.bz2 samba-e4e3293b482d48bc6a503e1280fe2a20556548ef.zip | |
s3-net: Check the return value of strlower_m().
Found by Coverity.
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
Diffstat (limited to 'source3/utils')
| -rw-r--r-- | source3/utils/net_rpc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source3/utils/net_rpc.c b/source3/utils/net_rpc.c index 2ccc92876b..c112a15e76 100644 --- a/source3/utils/net_rpc.c +++ b/source3/utils/net_rpc.c @@ -369,7 +369,9 @@ static NTSTATUS rpc_oldjoin_internals(struct net_context *c, } fstrcpy(trust_passwd, lp_netbios_name()); - strlower_m(trust_passwd); + if (!strlower_m(trust_passwd)) { + return NT_STATUS_UNSUCCESSFUL; + } /* * Machine names can be 15 characters, but the max length on |