summaryrefslogtreecommitdiff
path: root/source3
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2002-11-03 14:13:43 +0000
committerAndrew Bartlett <abartlet@samba.org>2002-11-03 14:13:43 +0000
commita7f2feae31ec5d377b55cc11e655ca8c39ac7679 (patch)
tree0998ca3f6f8899e32c7e985ba3d383fbc6abfce1 /source3
parent281284c819d0452c5d5db530ecdbd1267cd4e22c (diff)
downloadsamba-a7f2feae31ec5d377b55cc11e655ca8c39ac7679.tar.gz
samba-a7f2feae31ec5d377b55cc11e655ca8c39ac7679.tar.bz2
samba-a7f2feae31ec5d377b55cc11e655ca8c39ac7679.zip
Extra little fix to vl's patch. Make sure the passdb and testparm messages
say exactly the same thing - in particular that we can algorithmic rid base == 1000, and use the BASE_RID macro to avoid the use of magic numbers. Andrew Bartlett (This used to be commit b70f2a8047ac549841bc103932b38951e9814186)
Diffstat (limited to 'source3')
-rw-r--r--source3/passdb/passdb.c8
-rw-r--r--source3/utils/testparm.c6
2 files changed, 9 insertions, 5 deletions
diff --git a/source3/passdb/passdb.c b/source3/passdb/passdb.c
index adc5eb8478..4ce5b93abd 100644
--- a/source3/passdb/passdb.c
+++ b/source3/passdb/passdb.c
@@ -509,9 +509,11 @@ static int algorithmic_rid_base(void)
rid_offset = lp_algorithmic_rid_base();
- if (rid_offset < 1000) {
- DEBUG(0, ("algorithmic rid base must be above 1000\n"));
- rid_offset = 1000;
+ if (rid_offset < BASE_RID) {
+ /* Try to prevent admin foot-shooting, we can't put algorithmic
+ rids below 1000, that's the 'well known RIDs' on NT */
+ DEBUG(0, ("'algorithmic rid base' must be equal to or above %ld\n", BASE_RID));
+ rid_offset = BASE_RID;
}
if (rid_offset & 1) {
DEBUG(0, ("algorithmic rid base must be even\n"));
diff --git a/source3/utils/testparm.c b/source3/utils/testparm.c
index f2636fdafb..c92692fda2 100644
--- a/source3/utils/testparm.c
+++ b/source3/utils/testparm.c
@@ -168,8 +168,10 @@ via the %%o substitution. With encrypted passwords this is not possible.\n", lp_
printf("'winbind separator = +' might cause problems with group membership.\n");
}
- if (lp_algorithmic_rid_base() < 1000) {
- printf("'algorithmic rid base' must be equal or above 1000.\n");
+ if (lp_algorithmic_rid_base() < BASE_RID) {
+ /* Try to prevent admin foot-shooting, we can't put algorithmic
+ rids below 1000, that's the 'well known RIDs' on NT */
+ printf("'algorithmic rid base' must be equal to or above %lu\n", BASE_RID);
}
if (lp_algorithmic_rid_base() & 1) {