krb5 commit [krb5-1.15]: Fix kadm5.acl error reporting
Greg Hudson
ghudson at mit.edu
Mon Jul 17 22:59:37 EDT 2017
https://github.com/krb5/krb5/commit/a27b0a9fedd4f1902546426aeef73998a234ece8
commit a27b0a9fedd4f1902546426aeef73998a234ece8
Author: Greg Hudson <ghudson at mit.edu>
Date: Mon Jun 19 11:30:38 2017 -0400
Fix kadm5.acl error reporting
In kadm5int_acl_get_line(), increment *lnp after skipping a blank or
comment line, so that kadm5int_acl_load_acl_file() correctly reports
the line number if it fails to parse a line.
In acl_syn_err_msg, use %.10s to limit the amount of the line included
in the error message, not %10s to left-pad it with spaces if it is
shorter than ten characters.
(cherry picked from commit 3efb19294012ad38ac3a95d407e739313838c68a)
ticket: 8588
version_fixed: 1.15.2
src/lib/kadm5/srv/server_acl.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/lib/kadm5/srv/server_acl.c b/src/lib/kadm5/srv/server_acl.c
index 59ed0b9..3c2844d 100644
--- a/src/lib/kadm5/srv/server_acl.c
+++ b/src/lib/kadm5/srv/server_acl.c
@@ -91,7 +91,7 @@ static const char *acl_line2long_msg = N_("%s: line %d too long, truncated");
static const char *acl_op_bad_msg = N_("Unrecognized ACL operation '%c' in "
"%s");
static const char *acl_syn_err_msg = N_("%s: syntax error at line %d "
- "<%10s...>");
+ "<%.10s...>");
static const char *acl_cantopen_msg = N_("%s while opening ACL file %s");
/*
@@ -107,9 +107,9 @@ kadm5int_acl_get_line(fp, lnp)
static int line_incr = 0;
static char acl_buf[BUFSIZ];
- *lnp += line_incr;
- line_incr = 0;
for (domore = 1; domore && !feof(fp); ) {
+ *lnp += line_incr;
+ line_incr = 0;
/* Copy in the line, with continuations */
for (i = 0; ((i < BUFSIZ) && !feof(fp)); i++) {
int byte;
More information about the cvs-krb5
mailing list