Wednesday, March 30, 2016

[389-commits] ldap/servers

ldap/servers/slapd/log.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 6858dfd409a1e7207668f9a789a74bc4e9613d6a
Author: William Brown <firstyear@redhat.com>
Date: Wed Mar 30 13:04:58 2016 +1000

Ticket 47982 - HR Log timers, regression fix for subsystem logging

Bug Description: In the introduction of HR log times subsystem was omitted
from the errors log.

Fix Description: Re-add the subsystem parameter to subsystem logging.

https://fedorahosted.org/389/ticket/47982

Author: wibrown

Review by: mreynolds (Thanks!)

diff --git a/ldap/servers/slapd/log.c b/ldap/servers/slapd/log.c
index 80ff53e..34ca10d 100644
--- a/ldap/servers/slapd/log.c
+++ b/ldap/servers/slapd/log.c
@@ -2289,7 +2289,12 @@ vslapd_log_error(

/* blen = strlen(buffer); */
/* This truncates again .... But we have the nice smprintf above! */
- PR_snprintf (buffer+blen, sizeof(buffer)-blen, "%s", vbuf);
+ if (subsystem == NULL) {
+ PR_snprintf (buffer+blen, sizeof(buffer)-blen, "%s", vbuf);
+ } else {
+ PR_snprintf (buffer+blen, sizeof(buffer)-blen, "%s - %s", subsystem, vbuf);
+ }
+
buffer[sizeof(buffer)-1] = '\0';

if (fp)

--
389 commits mailing list
389-commits@%(host_name)s
http://lists.fedoraproject.org/admin/lists/389-commits@lists.fedoraproject.org

No comments:

Post a Comment