On Fri, Dec 07, 2012 at 02:13:59PM -0800, Marco Porsch wrote: > + if (info->attrs[NL80211_ATTR_BEACON_INTERVAL]) { > + setup.beacon_interval = > + nla_get_u32(info->attrs[NL80211_ATTR_BEACON_INTERVAL]); > + if (setup.beacon_interval < 10 || setup.beacon_interval > 10000) > + return -EINVAL; > + } > + > + if (info->attrs[NL80211_ATTR_DTIM_PERIOD]) { > + setup.dtim_period = > + nla_get_u32(info->attrs[NL80211_ATTR_DTIM_PERIOD]); > + if (setup.dtim_period < 1 || setup.dtim_period > 100) > + return -EINVAL; > + } Indentation looks kind of messed up there, two tabs instead of one? -- Bob Copeland %% www.bobcopeland.com -- 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