You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by br...@apache.org on 2019/09/08 07:47:07 UTC

svn commit: r1866588 - /subversion/trunk/subversion/libsvn_fs_fs/verify.c

Author: brane
Date: Sun Sep  8 07:47:07 2019
New Revision: 1866588

URL: http://svn.apache.org/viewvc?rev=1866588&view=rev
Log:
Follow up to r1865987: Fix another printf format bug in FSFS.

* subversion/libsvn_fs_fs/verify.c (compare_p2l_to_rev):
   Cast entry::type to unsigned integer and use the %u format specifier.

Found by: danielsh

Modified:
    subversion/trunk/subversion/libsvn_fs_fs/verify.c

Modified: subversion/trunk/subversion/libsvn_fs_fs/verify.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/verify.c?rev=1866588&r1=1866587&r2=1866588&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/verify.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/verify.c Sun Sep  8 07:47:07 2019
@@ -681,10 +681,10 @@ compare_p2l_to_rev(svn_fs_t *fs,
                                      NULL,
                                      _("p2l index entry for revision r%ld"
                                        " at offset %s contains invalid item"
-                                       " type %d"),
+                                       " type %u"),
                                      start,
                                      apr_off_t_toa(pool, offset),
-                                     entry->type);
+                                     (unsigned int)entry->type);
 
           /* There can be only one changes entry and that has a fixed type
            * and item number.  Its presence and parse-ability will be checked
@@ -696,10 +696,10 @@ compare_p2l_to_rev(svn_fs_t *fs,
                                      _("p2l index entry for changes in"
                                        " revision r%ld is item"
                                        " %"APR_UINT64_T_FMT
-                                       " of type %d at offset %s"),
+                                       " of type %u at offset %s"),
                                      entry->item.revision,
                                      entry->item.number,
-                                     entry->type,
+                                     (unsigned int)entry->type,
                                      apr_off_t_toa(pool, offset));
 
           /* Check contents. */