nzxt-kraken: Incorrect order of failing in probe

liquidctl / liquidtux

Linux kernel hwmon drivers for AIO liquid coolers and other devices

Other

79 stars 14 forks source link

nzxt-kraken: Incorrect order of failing in probe #61

Closed aleksamagicka closed 9 months ago

aleksamagicka commented 9 months ago

Both nzxt-kraken drivers share similar _probe() code with the same mistake of calling hid_hw_stop() and hid_hw_close() even if their _start() and _open() counterparts returned an error.

(Discovered on LKML by reusing the code for the gigabyte_waterforce driver.)

aleksamagicka commented 9 months ago

Patchwork link of the kraken2 fix for completeness, should still fix in nzxt-kraken3.