[PATCH 07/11] xfsprogs: xfs_fsr: replace sprintf with snprintf to avoid buffer overflow
Dave Chinner
david at fromorbit.com
Thu Dec 3 00:07:01 CST 2015
On Wed, Dec 02, 2015 at 11:44:02PM -0600, Eric Sandeen wrote:
>
> it seems like the sprintfs in i.e. fsrall_cleanup() and tmp_init()
> might have the same problem, no?
>
> And then what happens if it is truncated to SMBUFSZ; at that point
> I think this needs error handling, if the string got truncated.
Might be easier to simply increase the size of SMBUFSZ so that
overrun is not possible?
-Dave
--
Dave Chinner
david at fromorbit.com
More information about the xfs
mailing list