summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Potter <tpot@samba.org>2001-07-23 03:05:01 +0000
committerTim Potter <tpot@samba.org>2001-07-23 03:05:01 +0000
commit10fa3f878b9be6a78c10888ef9e9c3967f8d7e70 (patch)
tree6c9bfa3605122458eed678e46077e302d7ad9815
parent5f1130746f2d519cabacd3059528148916dc554d (diff)
downloadsamba-10fa3f878b9be6a78c10888ef9e9c3967f8d7e70.tar.gz
samba-10fa3f878b9be6a78c10888ef9e9c3967f8d7e70.tar.bz2
samba-10fa3f878b9be6a78c10888ef9e9c3967f8d7e70.zip
Added a warning debug if cli_samr_lookup_rids is called with more than 1000
rids as this seems to crash LSASS.EXE more often than not. (This used to be commit 375636b7630d117da5a57b51e11929c3a38646df)
-rw-r--r--source3/libsmb/cli_samr.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/source3/libsmb/cli_samr.c b/source3/libsmb/cli_samr.c
index 29f136427d..d8b300d090 100644
--- a/source3/libsmb/cli_samr.c
+++ b/source3/libsmb/cli_samr.c
@@ -792,7 +792,8 @@ uint32 cli_samr_query_dispinfo(struct cli_state *cli, TALLOC_CTX *mem_ctx,
return result;
}
-/* Lookup rids */
+/* Lookup rids. Note that NT4 seems to crash if more than ~1000 rids are
+ looked up in one packet. */
uint32 cli_samr_lookup_rids(struct cli_state *cli, TALLOC_CTX *mem_ctx,
POLICY_HND *domain_pol, uint32 flags,
@@ -805,6 +806,11 @@ uint32 cli_samr_lookup_rids(struct cli_state *cli, TALLOC_CTX *mem_ctx,
SAMR_R_LOOKUP_RIDS r;
uint32 result = NT_STATUS_UNSUCCESSFUL, i;
+ if (num_rids > 1000) {
+ DEBUG(2, ("cli_samr_lookup_rids: warning: NT4 can crash if "
+ "more than ~1000 rids are looked up at once.\n"));
+ }
+
ZERO_STRUCT(q);
ZERO_STRUCT(r);