[src/trunk]: src/distrib/cdrom/macppc_installboot PR/50915: David Binderman: ...

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/21c691a918a5
branches:  trunk
changeset: 344002:21c691a918a5
user:      christos <christos%NetBSD.org@localhost>
date:      Tue Mar 08 20:16:56 2016 +0000

description:
PR/50915: David Binderman: Fix infinite loop and printf format.

diffstat:

 distrib/cdrom/macppc_installboot/cd9660_util.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (27 lines):

diff -r 178c4fb85058 -r 21c691a918a5 distrib/cdrom/macppc_installboot/cd9660_util.c
--- a/distrib/cdrom/macppc_installboot/cd9660_util.c    Tue Mar 08 20:13:44 2016 +0000
+++ b/distrib/cdrom/macppc_installboot/cd9660_util.c    Tue Mar 08 20:16:56 2016 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cd9660_util.c,v 1.3 2014/09/27 15:21:40 tsutsui Exp $  */
+/*     $NetBSD: cd9660_util.c,v 1.4 2016/03/08 20:16:56 christos Exp $ */
 
 /*-
  * Copyright (c) 1994
@@ -99,7 +99,7 @@
        const u_char *isoend = isofn + isolen;
 
 #ifdef DEBUG
-       printf("fn = %s, fnlen = %d, isofn = %s, isolen = %d\n",
+       printf("fn = %s, fnlen = %zu, isofn = %s, isolen = %zu\n",
            fn, fnlen, isofn, isolen);
 #endif
 
@@ -119,7 +119,7 @@
                                break;
                        }
                        fn++;
-                       for (i = 0; --fnlen >= 0; i = i * 10 + *fn++ - '0') {
+                       for (i = 0; fnlen-- > 0; i = i * 10 + *fn++ - '0') {
                                if (*fn < '0' || *fn > '9') {
                                        return -1;
                                }