Smatch (http://repo.or.cz/w/smatch.git) complains about unreachable code in airo_set_auth() from drivers/net/wireless/airo.c We don't actually commit the changes, but instead break on line 6577. I didn't know what was intended there. Or who the maintainer was. regards, dan carpenter 6567 case IW_AUTH_80211_AUTH_ALG: { 6568 /* FIXME: What about AUTH_OPEN? This API seems to 6569 * disallow setting our auth to AUTH_OPEN. 6570 */ 6571 if (param->value & IW_AUTH_ALG_SHARED_KEY) { 6572 local->config.authType = AUTH_SHAREDKEY; 6573 } else if (param->value & IW_AUTH_ALG_OPEN_SYSTEM) { 6574 local->config.authType = AUTH_ENCRYPT; 6575 } else 6576 return -EINVAL; 6577 break; 6578 6579 /* Commit the changes to flags if needed */ 6580 if (local->config.authType != currentAuthType) 6581 set_bit (FLAG_COMMIT, &local->flags); 6582 } -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html