orinoco: fix checking for default value
authorAndrzej Hajda <a.hajda@samsung.com>
Mon, 21 Sep 2015 13:33:54 +0000 (15:33 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 22 Sep 2015 23:14:32 +0000 (16:14 -0700)
Thresholds uses -1 to indicate that default value should be used.
Since thresholds are unsigned sign checking makes no sense.

The problem has been detected using proposed semantic patch
scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1].

[1]: http://permalink.gmane.org/gmane.linux.kernel/2038576

Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/wireless/orinoco/cfg.c

index a9e94b6db5b7e961d25ec2c58a2eb50d46bc621d..0f6ea316e38ed419ba0ef5ca88d14284a8f84029 100644 (file)
@@ -220,7 +220,7 @@ static int orinoco_set_wiphy_params(struct wiphy *wiphy, u32 changed)
        if (changed & WIPHY_PARAM_FRAG_THRESHOLD) {
                /* Set fragmentation */
                if (priv->has_mwo) {
-                       if (wiphy->frag_threshold < 0)
+                       if (wiphy->frag_threshold == -1)
                                frag_value = 0;
                        else {
                                printk(KERN_WARNING "%s: Fixed fragmentation "
@@ -230,7 +230,7 @@ static int orinoco_set_wiphy_params(struct wiphy *wiphy, u32 changed)
                                frag_value = 1;
                        }
                } else {
-                       if (wiphy->frag_threshold < 0)
+                       if (wiphy->frag_threshold == -1)
                                frag_value = 2346;
                        else if ((wiphy->frag_threshold < 257) ||
                                 (wiphy->frag_threshold > 2347))
@@ -252,7 +252,7 @@ static int orinoco_set_wiphy_params(struct wiphy *wiphy, u32 changed)
                 * the upper limit.
                 */
 
-               if (wiphy->rts_threshold < 0)
+               if (wiphy->rts_threshold == -1)
                        rts_value = 2347;
                else if (wiphy->rts_threshold > 2347)
                        err = -EINVAL;