Skip to content

drivers: sensor: fix sensor_handlers checks #91209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 9, 2025

Conversation

kartben
Copy link
Collaborator

@kartben kartben commented Jun 6, 2025

z_vrfy_sensor_reconfigure_read_iodev function was performing a K_SYSCALL_MEMORY_READ check using an incorrect structure type.

z_vrfy_sensor_get_decoder wasn't properly checking output parameter decoder

@kartben kartben force-pushed the sensor_handlers branch from f8b1aa1 to 5b86b6c Compare June 6, 2025 20:51
@kartben kartben marked this pull request as ready for review June 6, 2025 20:52
@github-actions github-actions bot added size: XS A PR changing only a single line of code area: Sensors Sensors labels Jun 6, 2025
MaureenHelm
MaureenHelm previously approved these changes Jun 6, 2025
faxe1008
faxe1008 previously approved these changes Jun 7, 2025
@kartben kartben force-pushed the sensor_handlers branch from 5b86b6c to 771c94b Compare June 7, 2025 21:03
@kartben kartben changed the title drivers: sensor: fix K_SYSCALL_MEMORY_READ typo drivers: sensor: fix sensor_handlers checks Jun 7, 2025
z_vrfy_sensor_reconfigure_read_iodev function was performing a
a K_SYSCALL_MEMORY_READ check using an incorrect structure type.

z_vrfy_sensor_get_decoder wasn't properly checking output
parameter `decoder`

Signed-off-by: Benjamin Cabé <[email protected]>
@kartben kartben dismissed stale reviews from faxe1008 and MaureenHelm via aa98151 June 7, 2025 21:05
@kartben kartben force-pushed the sensor_handlers branch from 771c94b to aa98151 Compare June 7, 2025 21:05
Copy link

sonarqubecloud bot commented Jun 7, 2025

@kartben
Copy link
Collaborator Author

kartben commented Jun 7, 2025

spotted another mistake in the same file

@fabiobaltieri fabiobaltieri added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Jun 9, 2025
@dkalowsk dkalowsk merged commit 854713c into zephyrproject-rtos:main Jun 9, 2025
29 checks passed
@kartben kartben deleted the sensor_handlers branch June 9, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Sensors Sensors size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants