]> www.infradead.org Git - users/jedix/linux-maple.git/commit
media: cec: use us_to_ktime() where appropriate
authorVitaliy Shevtsov <v.shevtsov@mt-integration.ru>
Wed, 5 Mar 2025 15:11:25 +0000 (20:11 +0500)
committerHans Verkuil <hverkuil@xs4all.nl>
Thu, 6 Mar 2025 16:28:25 +0000 (17:28 +0100)
commitc0c1a6bf80e9075e3f6b81fd542550d8eb91e57a
tree292b5f356ff6ba4c9ec03f08321c2f6a6407feda
parente49563c3be09d4639ccff8ca68ef63b6f6c8456d
media: cec: use us_to_ktime() where appropriate

[Why]
There are several ns_to_ktime() calls that require using nanoseconds. It is
better to replace them with us_to_ktime() to make code clear, getting rid
of multiplication by 1000.

Also the timer function code may have an integer wrap-around issue. Since
both tx_custom_low_usecs and tx_custom_high_usecs can be set to up to
9999999 from the user space via cec_pin_error_inj_parse_line(), this may
cause usecs to be overflowed when adap->monitor_pin_cnt is zero and usecs
is multiplied by 1000.

[How]
Take advantage of using an appropriate helper func us_to_ktime() instead of
ns_to_ktime() to improve readability and to make the code clearer. And this
also mitigates possible integer wrap-arounds when usecs value is too large
and it is multiplied by 1000.

Found by Linux Verification Center (linuxtesting.org) with Svace.

Signed-off-by: Vitaliy Shevtsov <v.shevtsov@mt-integration.ru>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
drivers/media/cec/core/cec-pin.c