sonic-linux-kernel icon indicating copy to clipboard operation
sonic-linux-kernel copied to clipboard

Fix os crash caused by optoe when class switch

Open micas-net opened this issue 1 year ago • 4 comments

Why: When the device is initially set to optoe2, there is an option ->client [1]. If echo 3>dev_class is manually set at this time, as for optoe3, option ->client [1] is meaningless but the content is still the address data of optoe2, accessing an illegal address will occur, causing the device to crash.

How: Set option->client[1] to NULL when unregistering device

Closes:https://github.com/sonic-net/sonic-linux-kernel/issues/412

micas-net avatar Jul 02 '24 06:07 micas-net

Issue link:https://github.com/sonic-net/sonic-linux-kernel/issues/412

Please use:

Closes: https://github.com/sonic-net/sonic-linux-kernel/issues/412

paulmenzel avatar Jul 02 '24 06:07 paulmenzel

Could you please add the description to the commit message (git commit --amend && git push -f).

paulmenzel avatar Jul 02 '24 06:07 paulmenzel

Could you please add the description to the commit message (git commit --amend && git push -f).

@paulmenzel @lguohan @prgeor Ok,thanks, has updated. Help arrange review for this PR, thanks!

micas-net avatar Jul 03 '24 02:07 micas-net

As mentioned in issue:https://github.com/sonic-net/sonic-buildimage/issues/19525

micas-net avatar Jul 10 '24 07:07 micas-net