[PATCH v2] perf: xgene: Remove bogus IS_ERR() check
Al Viro
viro at ZenIV.linux.org.uk
Thu Oct 13 11:18:37 PDT 2016
On Thu, Oct 13, 2016 at 11:09:16AM -0700, Tai Nguyen wrote:
> In acpi_get_pmu_hw_inf we pass the address of a local variable to IS_ERR(),
> which doesn't make sense, as the pointer must be a real, valid pointer.
> This doesn't cause a functional problem, as IS_ERR() will evaluate as
> false, but the check is bogus and causes static checkers to complain.
... unless the test is actually a misspelled IS_ERR(res) and the current
code is broken by effectively skipping it.
More information about the linux-arm-kernel
mailing list