diff options
| author | Alex Elder <elder@inktank.com> | 2012-12-27 08:10:14 -0600 |
|---|---|---|
| committer | Ben Myers <bpm@sgi.com> | 2013-01-04 16:04:04 -0600 |
| commit | 59ad060e8f8c39406838c37360efb5f6c09a9041 (patch) | |
| tree | 3a16b41336dbc3bc9b18cde958994282827d17c8 | |
| parent | 877559d1bca3a4b649cec0b44beb7a972efae123 (diff) | |
| download | xfsdump-dev-59ad060e8f8c39406838c37360efb5f6c09a9041.tar.gz | |
xfsdump: fix format string in restore_spec()
Nigel Tamplin reported getting a seg fault in xfsrestore when a path
name was too long. He correctly diagnosed that the problem was due
to an extra "%s" format specifier in the format value passed to a
call to mlog(). This patch corrects that.
Signed-off-by: Alex Elder <elder@inktank.com>
Reported-by: Nigel Tamplin <ntamplin@codefaber.co.uk>
Tested-by: Nigel Tamplin <ntamplin@codefaber.co.uk>
Signed-off-by: Ben Myers <bpm@sgi.com>
| -rw-r--r-- | restore/content.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/restore/content.c b/restore/content.c index edd00ed2..54d933ca 100644 --- a/restore/content.c +++ b/restore/content.c @@ -7796,7 +7796,7 @@ restore_spec( filehdr_t *fhdrp, rv_t *rvp, char *path ) if ( strlen( path ) >= sizeof( addr.sun_path )) { mlog( MLOG_VERBOSE | MLOG_WARNING, _( "pathname too long for bind of " - "%s ino %llu %s: %s: discarding\n"), + "%s ino %llu %s: discarding\n"), printstr, fhdrp->fh_stat.bs_ino, path ); |