Thursday, June 1, 2017

[389-commits] [389-ds-base] 01/01: Ticket 49273 - Fix compiler warning in dbversion_write, missing newline

This is an automated email from the git hooks/post-receive script.

firstyear pushed a commit to branch master
in repository 389-ds-base.

commit 5d5b9c617e8c73f3771720c1f6906bddae2612db
Author: William Brown <firstyear@redhat.com>
Date: Thu Jun 1 09:34:41 2017 +1000

Ticket 49273 - Fix compiler warning in dbversion_write, missing newline

Bug Description: 49273 was missing a new line, and there was a
nearby (unrelated) compiler warning.

Fix Description: Add the new line, and resolve the compiler warning.

https://pagure.io/389-ds-base/issue/49273

Author: wibrown

Review by: mreynolds (Thanks!)
---
ldap/servers/slapd/back-ldbm/dbversion.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/ldap/servers/slapd/back-ldbm/dbversion.c b/ldap/servers/slapd/back-ldbm/dbversion.c
index 2d7d82e..33ca329 100644
--- a/ldap/servers/slapd/back-ldbm/dbversion.c
+++ b/ldap/servers/slapd/back-ldbm/dbversion.c
@@ -101,8 +101,7 @@ dbversion_write(struct ldbminfo *li, const char *directory,
len = strlen(buf);
if ( slapi_write_buffer( prfd, buf, len ) != (PRInt32)len )
{
- slapi_log_err(SLAPI_LOG_ERR, "dbversion_write",
- "Could not write to file \"%s\"\n", filename, 0, 0 );
+ slapi_log_err(SLAPI_LOG_ERR, "dbversion_write", "Could not write to file \"%s\"\n", filename );
rc= -1;
}
if(rc==0 && dataversion!=NULL)
@@ -111,8 +110,7 @@ dbversion_write(struct ldbminfo *li, const char *directory,
len = strlen( buf );
if ( slapi_write_buffer( prfd, buf, len ) != (PRInt32)len )
{
- slapi_log_err(SLAPI_LOG_ERR, "dbversion_write",
- "Could not write to file \"%s\"\n", filename, 0, 0 );
+ slapi_log_err(SLAPI_LOG_ERR, "dbversion_write", "Could not write to file \"%s\"\n", filename );
rc= -1;
}
}
@@ -185,7 +183,7 @@ dbversion_read(struct ldbminfo *li, const char *directory,
* which seems appropriate for the error here :)
*/
slapi_log_err(SLAPI_LOG_CRIT, "dbversion_read", "Could not parse file \"%s\". It may be corrupted.\n", filename);
- slapi_log_err(SLAPI_LOG_CRIT, "dbversion_read", "It may be possible to recover by replacing with a valid DBVERSION file from another DB instance");
+ slapi_log_err(SLAPI_LOG_CRIT, "dbversion_read", "It may be possible to recover by replacing with a valid DBVERSION file from another DB instance\n");
return EIDRM;
}


--
To stop receiving notification emails like this one, please contact
the administrator of this repository.
_______________________________________________
389-commits mailing list -- 389-commits@lists.fedoraproject.org
To unsubscribe send an email to 389-commits-leave@lists.fedoraproject.org

No comments:

Post a Comment