* [PATCH] xfsprogs: fix various incorrect printf formats
@ 2011-10-25 11:52 Christoph Hellwig
2011-10-25 15:59 ` Jakub Bogusz
0 siblings, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2011-10-25 11:52 UTC (permalink / raw)
To: xfs; +Cc: Jakub Bogusz
Reported-by: Jakub Bogusz <qboosh@pld-linux.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Index: xfsprogs/libxfs/freebsd.c
===================================================================
--- xfsprogs.orig/libxfs/freebsd.c 2011-10-25 13:44:11.614603520 +0200
+++ xfsprogs/libxfs/freebsd.c 2011-10-25 13:44:48.465103324 +0200
@@ -126,15 +126,13 @@ platform_findsizes(char *path, int fd, l
}
if ((st.st_mode & S_IFMT) != S_IFCHR) {
- fprintf(stderr, _("%s: "
- "Not a device or file: \"%s\"n"),
+ fprintf(stderr, _("%s: Not a device or file: \"%s\"\n"),
progname, path);
exit(1);
}
if (ioctl(fd, DIOCGMEDIASIZE, &size) != 0) {
- fprintf(stderr, _("%s: "
- "DIOCGMEDIASIZE failed on \"%s\": %s\n"),
+ fprintf(stderr, _("%s: DIOCGMEDIASIZE failed on \"%s\": %s\n"),
progname, path, strerror(errno));
exit(1);
}
Index: xfsprogs/repair/dinode.c
===================================================================
--- xfsprogs.orig/repair/dinode.c 2011-10-25 13:43:54.206603804 +0200
+++ xfsprogs/repair/dinode.c 2011-10-25 13:44:06.421658714 +0200
@@ -1439,7 +1439,7 @@ _("mismatch between format (%d) and size
}
if (dino->di_format == XFS_DINODE_FMT_LOCAL) {
do_warn(
-_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "n"),
+_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "\n"),
dino->di_format,
(__int64_t)be64_to_cpu(dino->di_size), lino);
return 1;
Index: xfsprogs/repair/dir2.c
===================================================================
--- xfsprogs.orig/repair/dir2.c 2011-10-25 13:46:25.830602794 +0200
+++ xfsprogs/repair/dir2.c 2011-10-25 13:46:51.049603282 +0200
@@ -1174,7 +1174,7 @@ _("would have corrected entry count in d
ino, sfp->hdr.count, i);
} else {
do_warn(
-_("corrected entry count in directory %" PRIu64 "u, was %d, now %d\n"),
+_("corrected entry count in directory %" PRIu64 ", was %d, now %d\n"),
ino, sfp->hdr.count, i);
sfp->hdr.count = i;
*dino_dirty = 1;
Index: xfsprogs/repair/phase6.c
===================================================================
--- xfsprogs.orig/repair/phase6.c 2011-10-25 13:44:53.826603054 +0200
+++ xfsprogs/repair/phase6.c 2011-10-25 13:46:21.041603352 +0200
@@ -2071,7 +2071,7 @@ longform_dir2_entry_check_data(
if (is_inode_free(irec, ino_offset)) {
nbad++;
if (entry_junked(
- _("entry \"%s\" in directory inode %" PRIu64 " points to free inode " PRIu64),
+ _("entry \"%s\" in directory inode %" PRIu64 " points to free inode %" PRIu64),
fname, ip->i_ino, inum)) {
dep->name[0] = '/';
libxfs_dir2_data_log_entry(tp, bp, dep);
Index: xfsprogs/repair/scan.c
===================================================================
--- xfsprogs.orig/repair/scan.c 2011-10-25 13:46:56.562602076 +0200
+++ xfsprogs/repair/scan.c 2011-10-25 13:47:31.369102715 +0200
@@ -289,14 +289,14 @@ _("bad back (left) sibling pointer (saw
*/
set_bmap(agno, agbno, XR_E_MULT);
do_warn(
-_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"),
+_("inode 0x%" PRIu64 "bmap block %" PRIx64 " claimed, state is %d\n"),
ino, bno, state);
break;
case XR_E_MULT:
case XR_E_INUSE_FS:
set_bmap(agno, agbno, XR_E_MULT);
do_warn(
-_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"),
+_("inode 0x%" PRIu64 " bmap block %" PRIx64 " claimed, state is %d\n"),
ino, bno, state);
/*
* if we made it to here, this is probably a bmap block
@@ -311,7 +311,7 @@ _("inode 0x%" PRIu64 " bmap block 0x%" P
case XR_E_BAD_STATE:
default:
do_warn(
-_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIu64 "\n"),
+_("bad state %d, inode 0x%" PRIu64 " bmap block %" PRIx64 "\n"),
state, ino, bno);
break;
}
@@ -338,7 +338,7 @@ _("bad state %d, inode 0x%" PRIu64 " bma
if (numrecs > mp->m_bmap_dmxr[0] || (isroot == 0 && numrecs <
mp->m_bmap_dmnr[0])) {
do_warn(
-_("inode 0x%" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"),
+_("inode %" PRIx64 " bad # of bmap records (%u, min - %u, max - %u)\n"),
ino, numrecs, mp->m_bmap_dmnr[0],
mp->m_bmap_dmxr[0]);
return(1);
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] xfsprogs: fix various incorrect printf formats 2011-10-25 11:52 [PATCH] xfsprogs: fix various incorrect printf formats Christoph Hellwig @ 2011-10-25 15:59 ` Jakub Bogusz 2011-11-02 9:20 ` [PATCH v2] " Christoph Hellwig 0 siblings, 1 reply; 5+ messages in thread From: Jakub Bogusz @ 2011-10-25 15:59 UTC (permalink / raw) To: Christoph Hellwig; +Cc: xfs On Tue, Oct 25, 2011 at 07:52:33AM -0400, Christoph Hellwig wrote: > Reported-by: Jakub Bogusz <qboosh@pld-linux.org> > Signed-off-by: Christoph Hellwig <hch@lst.de> [...] > Index: xfsprogs/repair/scan.c > =================================================================== > --- xfsprogs.orig/repair/scan.c 2011-10-25 13:46:56.562602076 +0200 > +++ xfsprogs/repair/scan.c 2011-10-25 13:47:31.369102715 +0200 > @@ -289,14 +289,14 @@ _("bad back (left) sibling pointer (saw > */ > set_bmap(agno, agbno, XR_E_MULT); > do_warn( > -_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"), > +_("inode 0x%" PRIu64 "bmap block %" PRIx64 " claimed, state is %d\n"), > ino, bno, state); > break; > case XR_E_MULT: > case XR_E_INUSE_FS: > set_bmap(agno, agbno, XR_E_MULT); > do_warn( > -_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"), > +_("inode 0x%" PRIu64 " bmap block %" PRIx64 " claimed, state is %d\n"), > ino, bno, state); > /* > * if we made it to here, this is probably a bmap block > @@ -311,7 +311,7 @@ _("inode 0x%" PRIu64 " bmap block 0x%" P > case XR_E_BAD_STATE: > default: > do_warn( > -_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIu64 "\n"), > +_("bad state %d, inode 0x%" PRIu64 " bmap block %" PRIx64 "\n"), > state, ino, bno); > break; > } > @@ -338,7 +338,7 @@ _("bad state %d, inode 0x%" PRIu64 " bma > if (numrecs > mp->m_bmap_dmxr[0] || (isroot == 0 && numrecs < > mp->m_bmap_dmnr[0])) { > do_warn( > -_("inode 0x%" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"), > +_("inode %" PRIx64 " bad # of bmap records (%u, min - %u, max - %u)\n"), > ino, numrecs, mp->m_bmap_dmnr[0], > mp->m_bmap_dmxr[0]); > return(1); Erm... the above part of changes is inconsistent. PRIxnn itself doesn't append "0x" - so either remove "0x" prefix and keep PRIu64 (thus printing value in decimal) _or_ (xor in terms of computer logic ;)) keep "0x" prefix and use PRIx64 (thus printing value in hex). I'd use decimal at least for inodes (in other places they are printed in such way). Also please note that in first 3 messages there were two uses "0x"+PRIu64... Regards, -- Jakub Bogusz http://qboosh.pl/ _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] xfsprogs: fix various incorrect printf formats 2011-10-25 15:59 ` Jakub Bogusz @ 2011-11-02 9:20 ` Christoph Hellwig 2011-11-05 9:00 ` Jakub Bogusz 0 siblings, 1 reply; 5+ messages in thread From: Christoph Hellwig @ 2011-11-02 9:20 UTC (permalink / raw) To: xfs; +Cc: Jakub Bogusz Reported-by: Jakub Bogusz <qboosh@pld-linux.org> Signed-off-by: Christoph Hellwig <hch@lst.de> Index: xfsprogs/libxfs/freebsd.c =================================================================== --- xfsprogs.orig/libxfs/freebsd.c 2011-10-30 05:56:36.180125966 +0100 +++ xfsprogs/libxfs/freebsd.c 2011-10-30 05:57:45.957126722 +0100 @@ -126,15 +126,13 @@ platform_findsizes(char *path, int fd, l } if ((st.st_mode & S_IFMT) != S_IFCHR) { - fprintf(stderr, _("%s: " - "Not a device or file: \"%s\"n"), + fprintf(stderr, _("%s: Not a device or file: \"%s\"\n"), progname, path); exit(1); } if (ioctl(fd, DIOCGMEDIASIZE, &size) != 0) { - fprintf(stderr, _("%s: " - "DIOCGMEDIASIZE failed on \"%s\": %s\n"), + fprintf(stderr, _("%s: DIOCGMEDIASIZE failed on \"%s\": %s\n"), progname, path, strerror(errno)); exit(1); } Index: xfsprogs/repair/dinode.c =================================================================== --- xfsprogs.orig/repair/dinode.c 2011-10-30 05:56:36.132124261 +0100 +++ xfsprogs/repair/dinode.c 2011-10-30 05:57:45.957126722 +0100 @@ -1439,7 +1439,7 @@ _("mismatch between format (%d) and size } if (dino->di_format == XFS_DINODE_FMT_LOCAL) { do_warn( -_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "n"), +_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "\n"), dino->di_format, (__int64_t)be64_to_cpu(dino->di_size), lino); return 1; Index: xfsprogs/repair/dir2.c =================================================================== --- xfsprogs.orig/repair/dir2.c 2011-10-30 05:56:36.156126568 +0100 +++ xfsprogs/repair/dir2.c 2011-11-02 10:16:46.593593447 +0100 @@ -1174,7 +1174,7 @@ _("would have corrected entry count in d ino, sfp->hdr.count, i); } else { do_warn( -_("corrected entry count in directory %" PRIu64 "u, was %d, now %d\n"), +_("corrected entry count in directory %" PRIu64 ", was %d, now %d\n"), ino, sfp->hdr.count, i); sfp->hdr.count = i; *dino_dirty = 1; Index: xfsprogs/repair/phase6.c =================================================================== --- xfsprogs.orig/repair/phase6.c 2011-10-30 05:56:36.144127454 +0100 +++ xfsprogs/repair/phase6.c 2011-10-30 05:57:45.973171205 +0100 @@ -2071,7 +2071,7 @@ longform_dir2_entry_check_data( if (is_inode_free(irec, ino_offset)) { nbad++; if (entry_junked( - _("entry \"%s\" in directory inode %" PRIu64 " points to free inode " PRIu64), + _("entry \"%s\" in directory inode %" PRIu64 " points to free inode %" PRIu64), fname, ip->i_ino, inum)) { dep->name[0] = '/'; libxfs_dir2_data_log_entry(tp, bp, dep); Index: xfsprogs/repair/scan.c =================================================================== --- xfsprogs.orig/repair/scan.c 2011-10-30 05:56:36.168127218 +0100 +++ xfsprogs/repair/scan.c 2011-10-30 05:57:45.981169953 +0100 @@ -289,14 +289,14 @@ _("bad back (left) sibling pointer (saw */ set_bmap(agno, agbno, XR_E_MULT); do_warn( -_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"), +_("inode 0x%" PRIu64 "bmap block 0x%" PRIx64 " claimed, state is %d\n"), ino, bno, state); break; case XR_E_MULT: case XR_E_INUSE_FS: set_bmap(agno, agbno, XR_E_MULT); do_warn( -_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"), +_("inode 0x%" PRIu64 " bmap block 0x%" PRIx64 " claimed, state is %d\n"), ino, bno, state); /* * if we made it to here, this is probably a bmap block @@ -311,7 +311,7 @@ _("inode 0x%" PRIu64 " bmap block 0x%" P case XR_E_BAD_STATE: default: do_warn( -_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIu64 "\n"), +_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIx64 "\n"), state, ino, bno); break; } @@ -338,7 +338,7 @@ _("bad state %d, inode 0x%" PRIu64 " bma if (numrecs > mp->m_bmap_dmxr[0] || (isroot == 0 && numrecs < mp->m_bmap_dmnr[0])) { do_warn( -_("inode 0x%" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"), +_("inode %" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"), ino, numrecs, mp->m_bmap_dmnr[0], mp->m_bmap_dmxr[0]); return(1); _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] xfsprogs: fix various incorrect printf formats 2011-11-02 9:20 ` [PATCH v2] " Christoph Hellwig @ 2011-11-05 9:00 ` Jakub Bogusz 2011-11-05 9:33 ` Christoph Hellwig 0 siblings, 1 reply; 5+ messages in thread From: Jakub Bogusz @ 2011-11-05 9:00 UTC (permalink / raw) To: xfs; +Cc: Christoph Hellwig [-- Attachment #1: Type: text/plain, Size: 1140 bytes --] On Wed, Nov 02, 2011 at 05:20:12AM -0400, Christoph Hellwig wrote: > Index: xfsprogs/repair/scan.c > =================================================================== > --- xfsprogs.orig/repair/scan.c 2011-10-30 05:56:36.168127218 +0100 > +++ xfsprogs/repair/scan.c 2011-10-30 05:57:45.981169953 +0100 > @@ -289,14 +289,14 @@ _("bad back (left) sibling pointer (saw > */ > set_bmap(agno, agbno, XR_E_MULT); > do_warn( > -_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"), > +_("inode 0x%" PRIu64 "bmap block 0x%" PRIx64 " claimed, state is %d\n"), > ino, bno, state); > break; > case XR_E_MULT: > case XR_E_INUSE_FS: > set_bmap(agno, agbno, XR_E_MULT); > do_warn( > -_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"), > +_("inode 0x%" PRIu64 " bmap block 0x%" PRIx64 " claimed, state is %d\n"), > ino, bno, state); > /* > * if we made it to here, this is probably a bmap block These two entries still need fix in the first format strings of both msgids... I'm attaching my version of the whole patch. -- Jakub Bogusz http://qboosh.pl/ [-- Attachment #2: xfsprogs-msgids.patch --] [-- Type: text/plain, Size: 4227 bytes --] Reported-by: Jakub Bogusz <qboosh@pld-linux.org> Signed-off-by: Christoph Hellwig <hch@lst.de> Index: xfsprogs/libxfs/freebsd.c =================================================================== --- xfsprogs.orig/libxfs/freebsd.c 2011-10-30 05:56:36.180125966 +0100 +++ xfsprogs/libxfs/freebsd.c 2011-10-30 05:57:45.957126722 +0100 @@ -126,15 +126,13 @@ platform_findsizes(char *path, int fd, l } if ((st.st_mode & S_IFMT) != S_IFCHR) { - fprintf(stderr, _("%s: " - "Not a device or file: \"%s\"n"), + fprintf(stderr, _("%s: Not a device or file: \"%s\"\n"), progname, path); exit(1); } if (ioctl(fd, DIOCGMEDIASIZE, &size) != 0) { - fprintf(stderr, _("%s: " - "DIOCGMEDIASIZE failed on \"%s\": %s\n"), + fprintf(stderr, _("%s: DIOCGMEDIASIZE failed on \"%s\": %s\n"), progname, path, strerror(errno)); exit(1); } Index: xfsprogs/repair/dinode.c =================================================================== --- xfsprogs.orig/repair/dinode.c 2011-10-30 05:56:36.132124261 +0100 +++ xfsprogs/repair/dinode.c 2011-10-30 05:57:45.957126722 +0100 @@ -1439,7 +1439,7 @@ _("mismatch between format (%d) and size } if (dino->di_format == XFS_DINODE_FMT_LOCAL) { do_warn( -_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "n"), +_("mismatch between format (%d) and size (%" PRId64 ") in symlink inode %" PRIu64 "\n"), dino->di_format, (__int64_t)be64_to_cpu(dino->di_size), lino); return 1; Index: xfsprogs/repair/dir2.c =================================================================== --- xfsprogs.orig/repair/dir2.c 2011-10-30 05:56:36.156126568 +0100 +++ xfsprogs/repair/dir2.c 2011-11-02 10:16:46.593593447 +0100 @@ -1174,7 +1174,7 @@ _("would have corrected entry count in d ino, sfp->hdr.count, i); } else { do_warn( -_("corrected entry count in directory %" PRIu64 "u, was %d, now %d\n"), +_("corrected entry count in directory %" PRIu64 ", was %d, now %d\n"), ino, sfp->hdr.count, i); sfp->hdr.count = i; *dino_dirty = 1; Index: xfsprogs/repair/phase6.c =================================================================== --- xfsprogs.orig/repair/phase6.c 2011-10-30 05:56:36.144127454 +0100 +++ xfsprogs/repair/phase6.c 2011-10-30 05:57:45.973171205 +0100 @@ -2071,7 +2071,7 @@ longform_dir2_entry_check_data( if (is_inode_free(irec, ino_offset)) { nbad++; if (entry_junked( - _("entry \"%s\" in directory inode %" PRIu64 " points to free inode " PRIu64), + _("entry \"%s\" in directory inode %" PRIu64 " points to free inode %" PRIu64), fname, ip->i_ino, inum)) { dep->name[0] = '/'; libxfs_dir2_data_log_entry(tp, bp, dep); Index: xfsprogs/repair/scan.c =================================================================== --- xfsprogs.orig/repair/scan.c 2011-10-30 05:56:36.168127218 +0100 +++ xfsprogs/repair/scan.c 2011-10-30 05:57:45.981169953 +0100 @@ -289,14 +289,14 @@ _("bad back (left) sibling pointer (saw */ set_bmap(agno, agbno, XR_E_MULT); do_warn( -_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"), +_("inode %" PRIu64 "bmap block 0x%" PRIx64 " claimed, state is %d\n"), ino, bno, state); break; case XR_E_MULT: case XR_E_INUSE_FS: set_bmap(agno, agbno, XR_E_MULT); do_warn( -_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"), +_("inode %" PRIu64 " bmap block 0x%" PRIx64 " claimed, state is %d\n"), ino, bno, state); /* * if we made it to here, this is probably a bmap block @@ -311,7 +311,7 @@ _("inode 0x%" PRIu64 " bmap block 0x%" P case XR_E_BAD_STATE: default: do_warn( -_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIu64 "\n"), +_("bad state %d, inode 0x%" PRIu64 " bmap block 0x%" PRIx64 "\n"), state, ino, bno); break; } @@ -338,7 +338,7 @@ _("bad state %d, inode 0x%" PRIu64 " bma if (numrecs > mp->m_bmap_dmxr[0] || (isroot == 0 && numrecs < mp->m_bmap_dmnr[0])) { do_warn( -_("inode 0x%" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"), +_("inode %" PRIu64 " bad # of bmap records (%u, min - %u, max - %u)\n"), ino, numrecs, mp->m_bmap_dmnr[0], mp->m_bmap_dmxr[0]); return(1); [-- Attachment #3: Type: text/plain, Size: 121 bytes --] _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] xfsprogs: fix various incorrect printf formats 2011-11-05 9:00 ` Jakub Bogusz @ 2011-11-05 9:33 ` Christoph Hellwig 0 siblings, 0 replies; 5+ messages in thread From: Christoph Hellwig @ 2011-11-05 9:33 UTC (permalink / raw) To: Jakub Bogusz; +Cc: Christoph Hellwig, xfs > > do_warn( > > -_("inode 0x%" PRIu64 "bmap block 0x%" PRIu64 " claimed, state is %d\n"), > > +_("inode 0x%" PRIu64 "bmap block 0x%" PRIx64 " claimed, state is %d\n"), > > ino, bno, state); > > break; > > case XR_E_MULT: > > case XR_E_INUSE_FS: > > set_bmap(agno, agbno, XR_E_MULT); > > do_warn( > > -_("inode 0x%" PRIu64 " bmap block 0x%" PRIu64 " claimed, state is %d\n"), > > +_("inode 0x%" PRIu64 " bmap block 0x%" PRIx64 " claimed, state is %d\n"), > > ino, bno, state); > > /* > > * if we made it to here, this is probably a bmap block > > These two entries still need fix in the first format strings of both > msgids... > > I'm attaching my version of the whole patch. I've already put the previous version in. I'll commit the updates attributed to you. Sorry for the mess, and thanks a lot for looking over this. _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-11-05 9:33 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-10-25 11:52 [PATCH] xfsprogs: fix various incorrect printf formats Christoph Hellwig 2011-10-25 15:59 ` Jakub Bogusz 2011-11-02 9:20 ` [PATCH v2] " Christoph Hellwig 2011-11-05 9:00 ` Jakub Bogusz 2011-11-05 9:33 ` Christoph Hellwig
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox