Re: [PATCH] mei: fix incorrect logical operator in if statement

2017-12-20 Thread Colin Ian King
On 20/12/17 08:58, Winkler, Tomas wrote: > >> From: Colin Ian King >> >> The current expression using the || operator is always true because >> dev->dev_state cannot be equal to two different values at the same time. >> Fix this by replacing the || with &&. >> >> Detected by CoverityScan, CID#146

RE: [PATCH] mei: fix incorrect logical operator in if statement

2017-12-20 Thread Winkler, Tomas
> From: Colin Ian King > > The current expression using the || operator is always true because > dev->dev_state cannot be equal to two different values at the same time. > Fix this by replacing the || with &&. > > Detected by CoverityScan, CID#1463042 ("Constant expression result") > > Fixes: