mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-27 04:47:05 +00:00
iio: addac: ad74413r: fix integer promotion bug in ad74413_get_input_current_offset()
commit980389d06d
upstream. The constant AD74413R_ADC_RESULT_MAX is defined via GENMASK, so its type is "unsigned long". Hence in the expression voltage_offset * AD74413R_ADC_RESULT_MAX, voltage_offset is first promoted to unsigned long, and since it may be negative, that results in a garbage value. For example, when range is AD74413R_ADC_RANGE_5V_BI_DIR, voltage_offset is -2500 and voltage_range is 5000, so the RHS of this assignment is, depending on sizeof(long), either 826225UL or 3689348814709142UL, which after truncation to int then results in either 826225 or 1972216214 being the output from in_currentX_offset. Casting to int avoids that promotion and results in the correct -32767 output. Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk> Fixes:fea251b6a5
(iio: addac: add AD74413R driver) Reviewed-by: Nuno Sá <nuno.sa@analog.com> Link: https://lore.kernel.org/r/20221118123209.1658420-1-linux@rasmusvillemoes.dk Cc: <Stable@vger.kernel.org> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
b1768dd348
commit
d8c1111add
1 changed files with 1 additions and 1 deletions
|
@ -691,7 +691,7 @@ static int ad74413_get_input_current_offset(struct ad74413r_state *st,
|
|||
if (ret)
|
||||
return ret;
|
||||
|
||||
*val = voltage_offset * AD74413R_ADC_RESULT_MAX / voltage_range;
|
||||
*val = voltage_offset * (int)AD74413R_ADC_RESULT_MAX / voltage_range;
|
||||
|
||||
return IIO_VAL_INT;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue