[PATCH][TRIVIAL] Unvalid access check

[PATCH][TRIVIAL] Unvalid access check

[Goffredo Baroncelli]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Dear All,

I found an incorrect check in the file fs/btrfs/send.c . It was wrote
	sizeof(*arg->clone_sources * arg->clone_sources_count)
instead of
	sizeof(*arg->clone_sources) * arg->clone_sources_count

Unfortunately arg->clone_sources is a u64 pointer, so
"*arg->clone_sources * arg->clone_sources_count" is a legally product
between two u64 integer, and the compiler cannot generate any
warning/error.

Please apply


BR
G.Baroncelli

- --- old/fs/btrfs/send.c	2012-12-11 19:03:46.000000000 +0100
+++ new/fs/btrfs/send.c	2012-12-17 18:42:14.997707719 +0100
@@ -4553,8 +4553,8 @@
 	}

 	if (!access_ok(VERIFY_READ, arg->clone_sources,
- -			sizeof(*arg->clone_sources *
- -			arg->clone_sources_count))) {
+			sizeof(*arg->clone_sources) *
+			arg->clone_sources_count)) {
 		ret = -EFAULT;
 		goto out;
 	}

- -- 
gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D  17B2 0EDA 9B37 8B82 E0B5
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJQz1yXAAoJEA7amzeLguC1UH8QAJozuS6LTzk94CSr0qqM70xa
QC9UgY3MjalIlFPBHqRKTcbId2uvtM0yqwk2YCHRAnKrMUAMvP1KmUlk5Fjg3ahC
odPbfX1eF3zf9ySEOky7ZmzjJEpyxsCikvf2CNaUiQSibjLn6bNYmhjjXKrIqmOR
KZAFGrEOzeK87YUREcqqaAz+oVZHCZuxHrA0OpWJ3L0ub9kC35o2zdEYJejwDUPk
zGwWmMhMr9ANvUMC/E8wNzusoZRnCya/dEt6KrN1Xzolj82orhQRHWy3TTUEtwu9
1zZjq89vVhUKNh4/CG7u0N5zl61ZhzGiXATpIEC2M/PznMf0YXmITi2Dy62fTwnT
ZvLjUmUnujmSGBMi1/zEoyYFXPmFgHfa82s0zVQCKC6V2JZcapwSbDSJaLYKjhjs
rjd6SFsjjEWkE779bFW1WHu0CV3BvXJI7yXN6PweR9rumhvU1pZ4UwonQ3Wv1tPp
d50N9jeFq2+N282yP12bdGzD7DLC7YPFRFxy8sHSbOsiM3vFIdOK0rX5BQeboIbE
NPBHPqUsxw/MsvXQdUqgW7TuFIU0Q30Mx0+jLyyhmGSvCUq9NA9ye0YmRCmqdGGY
R5X4KdL7HJxlm9tX2FxLPf6yh3hD/FyX1CQtrfahbvS0h2em5BLPQ7WrGN2iCpAd
cEcSAGaARv6qKnw+YneW
=kdsU
-----END PGP SIGNATURE-----