krb5 commit [krb5-1.15]: Fix kadm5.acl error reporting

Greg Hudson <[email protected]>
Newsgroups gmane.comp.encryption.kerberos.cvs
Message-ID <[email protected]>
https://github.com/krb5/krb5/commit/a27b0a9fedd4f1902546426aeef73998a234ece8
commit a27b0a9fedd4f1902546426aeef73998a234ece8
Author: Greg Hudson <[email protected]>
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;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.