Hi,
Thayumanavar Sachithanantham:
> correct name could not go beyond 242 chars and because of cnt/%d the
> buffer could be overrun.
> I think having
> snprintf(p, WH_TMP_LEN+1, "%.*x", AUFS_WH_TMP_LEN, cnt++); would be better.
'cnt' is 'short' and its size is guranteed as
BUILD_BUG_ON(sizeof(cnt) * 2 > AUFS_WH_TMP_LEN);
(earlier in the same function).
Is it really neccesary to specify WH_TMP_LEN+1?
J. R. Okajima
------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
