meta-starlingx/meta-stx-integ/recipes-support/openldap/files/0013-openldap-missing-unlock-in-accesslog-overlay.patch

   1 From 7cc8c2c22dc6a5999554e64b25f162b3673cd922 Mon Sep 17 00:00:00 2001
   2 From: babak sarashki <babak.sarashki@windriver.com>
   3 Date: Sun, 3 Nov 2019 14:48:29 -0800
   4 Subject: [PATCH 13/20] openldap missing unlock in accesslog overlay
   5
   6 From stx 1901: openldap-missing-unlock-in-accesslog-overlay.patch
   7 A mutex lock might not get unlocked when plausible
   8
   9 In the preceding if-statement a mutex may get locked. This is unlocked on
  10 'done' label, but not called when plausible. Based on the current code logic
  11 this seems to not be able to happen, but might when code gets changed at
  12 some point in time. This patch fixes the issue.
  13
  14 The issue was found by Coverity scan
  15 http://cov01.lab.eng.brq.redhat.com/covscanhub/waiving/11054/38577/
  16
  17 Author: Matus Honek <mhonek@redhat.com>
  18 Resolves: #1261003
  19 ---
  20  servers/slapd/overlays/accesslog.c | 2 +-
  21  1 file changed, 1 insertion(+), 1 deletion(-)
  22
  23 diff --git a/servers/slapd/overlays/accesslog.c b/servers/slapd/overlays/accesslog.c
  24 index 6162b57..f6d7fc1 100644
  25 --- a/servers/slapd/overlays/accesslog.c
  26 +++ b/servers/slapd/overlays/accesslog.c
  27 @@ -1517,7 +1517,7 @@ static int accesslog_response(Operation *op, SlapReply *rs) {
  28
  29         /* ignore these internal reads */
  30         if (( lo->mask & LOG_OP_READS ) && op->o_do_not_cache ) {
  31 -               return SLAP_CB_CONTINUE;
  32 +               goto done;
  33         }
  34
  35         if ( li->li_success && rs->sr_err != LDAP_SUCCESS )
  36 --
  37 2.17.1
  38