From 37bff7c59f8cca09ba2a2e6eb3e6c852327bbb17 Mon Sep 17 00:00:00 2001 From: Rich Megginson Date: Wed, 11 May 2005 18:51:54 +0000 Subject: Bug(s) fixed: 157377 Bug Description: RL16: Server core dumped while configuring replication agreements Reviewed by: David (Thanks!) Fix Description: The code for handling bad agreements was wrong. It was not returning the correct error code. The result of this is that the client did not receive an appropriate error code/message from the server. The reason for the core dump was that, even though the agreement was not created, the DSE entry was still there, causing the server to become very confused. Returning the correct error code causes the server to refuse to create the bad DSE entry. Platforms tested: RHEL3 Flag Day: no Doc impact: no QA impact: should be covered by regular nightly and manual testing New Tests integrated into TET: none --- ldap/servers/plugins/replication/repl5_agmtlist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ldap/servers/plugins/replication/repl5_agmtlist.c b/ldap/servers/plugins/replication/repl5_agmtlist.c index b8abe4c0..2206f83d 100644 --- a/ldap/servers/plugins/replication/repl5_agmtlist.c +++ b/ldap/servers/plugins/replication/repl5_agmtlist.c @@ -147,7 +147,7 @@ add_new_agreement(Slapi_Entry *e) Object *repl_obj = NULL; Object *ro = NULL; - if (ra == NULL) return 0; + if (ra == NULL) return 1; /* tell search result handler callback this entry was not sent */ ro = object_new((void *)ra, agmt_delete); objset_add_obj(agmt_set, ro); -- cgit