| commit | 436869e0cd6dd700a9d93c551d08401fd0a94d40 | |
| author | Will Deacon <will.deacon@arm.com> | |
| Wed, 10 Apr 2019 10:49:11 +0000 (10 11:49 +0100) | ||
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
| Wed, 3 Jul 2019 11:14:49 +0000 (3 13:14 +0200) | ||
| tree | cb29c6f0f60d173797e9e69d034483e211c4e00d | treesnapshot (tar.gz zip) |
| parent | ba6340a7297fdb36550fa7800500eadc8278c062 | commitdiff |
arm64: futex: Avoid copying out uninitialised stack in failed cmpxchg()
commit 8e4e0ac02b449297b86498ac24db5786ddd9f647 upstream.
Returning an error code from futex_atomic_cmpxchg_inatomic() indicates
that the caller should not make any use of *uval, and should instead act
upon on the value of the error code. Although this is implemented
correctly in our futex code, we needlessly copy uninitialised stack to
*uval in the error case, which can easily be avoided.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
commit 8e4e0ac02b449297b86498ac24db5786ddd9f647 upstream.
Returning an error code from futex_atomic_cmpxchg_inatomic() indicates
that the caller should not make any use of *uval, and should instead act
upon on the value of the error code. Although this is implemented
correctly in our futex code, we needlessly copy uninitialised stack to
*uval in the error case, which can easily be avoided.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| arch/arm64/include/asm/futex.h | diffblobblamehistory |