From 5ffe12c065c06ecf0ad26d59a465f7a95611fc9c Mon Sep 17 00:00:00 2001 From: Howard Chu Date: Thu, 6 Aug 2026 15:22:58 +0100 Subject: [PATCH] ITS#10559 back-sock: treat partial response as failure --- servers/slapd/back-sock/result.c | 3 +-- servers/slapd/result.c | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/servers/slapd/back-sock/result.c b/servers/slapd/back-sock/result.c index 7640eefee1..0baeb2f109 100644 --- a/servers/slapd/back-sock/result.c +++ b/servers/slapd/back-sock/result.c @@ -31,8 +31,7 @@ #include "back-sock.h" /* - * FIXME: make a RESULT section compulsory from the socket response. - * Otherwise, a partial/aborted response is treated as 'success'. + * Note: str2result treats a partial/aborted response as failure, LDAP_OTHER. * This is a divergence from the back-shell protocol, but makes things * more robust. */ diff --git a/servers/slapd/result.c b/servers/slapd/result.c index ad7abdf535..46304fa2df 100644 --- a/servers/slapd/result.c +++ b/servers/slapd/result.c @@ -1722,7 +1722,7 @@ str2result( int rc; char *c; - *code = LDAP_SUCCESS; + *code = LDAP_OTHER; *matched = NULL; *info = NULL; -- 2.47.3