projects / cascardo / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 655aa39) | patch
net: wimax: i2400m: control.c: Cleaning up conjunction always evaluates to false
authorRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Sat, 7 Jun 2014 11:26:37 +0000 (13:26 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 11 Jun 2014 07:13:16 +0000 (00:13 -0700)
commitf05d435bde588ac26ba75f7f6893b2a63ae71381
treefa4cf2b2306cdc957408e6a8eeedbfc3b3559109tree | snapshot
parent655aa393063b0cca03d1c4d1d2971b9b9770525bcommit | diff
net: wimax: i2400m: control.c: Cleaning up conjunction always evaluates to false

Logical conjunction always evaluates to false:  minor < 2 && minor > 1
I guess what you wanted is rather: minor > 2 || minor < 1

This was partly found using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/wimax/i2400m/control.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.