From cd83c1d9582e3252b2c964d77aebf1d241d371a6 Mon Sep 17 00:00:00 2001 From: Gregor Beck Date: Wed, 6 Jul 2011 15:10:27 +0200 Subject: s3:testparm: avoid spurious warnings about dos mode mapping and create mask if store dos attributes is set Autobuild-User: Volker Lendecke Autobuild-Date: Thu Jul 7 12:49:01 CEST 2011 on sn-devel-104 --- source3/utils/testparm.c | 46 +++++++++++++++++++++++++++------------------- 1 file changed, 27 insertions(+), 19 deletions(-) (limited to 'source3/utils') diff --git a/source3/utils/testparm.c b/source3/utils/testparm.c index 966b61e58e..9b224d4c1b 100644 --- a/source3/utils/testparm.c +++ b/source3/utils/testparm.c @@ -307,25 +307,33 @@ static void do_per_share_checks(int s) lp_servicename(s) ); } - if (lp_map_hidden(s) && !(lp_create_mask(s) & S_IXOTH)) { - fprintf(stderr,"Invalid combination of parameters for service %s. \ - Map hidden can only work if create mask includes octal 01 (S_IXOTH).\n", - lp_servicename(s) ); - } - if (lp_map_hidden(s) && (lp_force_create_mode(s) & S_IXOTH)) { - fprintf(stderr,"Invalid combination of parameters for service %s. \ - Map hidden can only work if force create mode excludes octal 01 (S_IXOTH).\n", - lp_servicename(s) ); - } - if (lp_map_system(s) && !(lp_create_mask(s) & S_IXGRP)) { - fprintf(stderr,"Invalid combination of parameters for service %s. \ - Map system can only work if create mask includes octal 010 (S_IXGRP).\n", - lp_servicename(s) ); - } - if (lp_map_system(s) && (lp_force_create_mode(s) & S_IXGRP)) { - fprintf(stderr,"Invalid combination of parameters for service %s. \ - Map system can only work if force create mode excludes octal 010 (S_IXGRP).\n", - lp_servicename(s) ); + if (!lp_store_dos_attributes(s) && lp_map_hidden(s) + && !(lp_create_mask(s) & S_IXOTH)) + { + fprintf(stderr,"Invalid combination of parameters for service " + "%s. Map hidden can only work if create mask includes " + "octal 01 (S_IXOTH).\n", lp_servicename(s)); + } + if (!lp_store_dos_attributes(s) && lp_map_hidden(s) + && (lp_force_create_mode(s) & S_IXOTH)) + { + fprintf(stderr,"Invalid combination of parameters for service " + "%s. Map hidden can only work if force create mode " + "excludes octal 01 (S_IXOTH).\n", lp_servicename(s)); + } + if (!lp_store_dos_attributes(s) && lp_map_system(s) + && !(lp_create_mask(s) & S_IXGRP)) + { + fprintf(stderr,"Invalid combination of parameters for service " + "%s. Map system can only work if create mask includes " + "octal 010 (S_IXGRP).\n", lp_servicename(s)); + } + if (!lp_store_dos_attributes(s) && lp_map_system(s) + && (lp_force_create_mode(s) & S_IXGRP)) + { + fprintf(stderr,"Invalid combination of parameters for service " + "%s. Map system can only work if force create mode " + "excludes octal 010 (S_IXGRP).\n", lp_servicename(s)); } #ifdef HAVE_CUPS if (lp_printing(s) == PRINT_CUPS && *(lp_printcommand(s)) != '\0') { -- cgit