param: correctly return false when parsing invalid octal in lp_do_parameter
authorGarming Sam <garming@catalyst.net.nz>
Tue, 25 Feb 2014 22:21:35 +0000 (11:21 +1300)
committerJeremy Allison <jra@samba.org>
Wed, 7 May 2014 17:49:17 +0000 (19:49 +0200)
Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/param/loadparm.c

index 844cfe903d15de1da9c6ecd181006a4af4ef7899..52dc57db40fba40ee494509d9c91a54562f0773c 100644 (file)
@@ -2659,6 +2659,7 @@ bool lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue
                        i = sscanf(pszParmValue, "%o", (int *)parm_ptr);
                        if ( i != 1 ) {
                            DEBUG ( 0, ("Invalid octal number %s\n", pszParmName ));
+                               return false;
                        }
                        break;