On Mon, Jan 24, 2011 at 13:38, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
> On Sun, 2011-01-23 at 23:02 +0200, Arik Nemtsov wrote:
>
>> +static int ieee80211_set_probe_resp(struct wiphy *wiphy,
>> + struct net_device *dev, u8 *resp,
>> + size_t resp_len)
>> +{
>> + struct ieee80211_sub_if_data *sdata;
>> + struct sk_buff *new, *old;
>> +
>> + sdata = IEEE80211_DEV_TO_SUB_IF(dev);
>> + old = sdata->u.ap.probe_resp;
>> +
>> + if (!resp || !resp_len)
>> + return -EINVAL;
>> +
>> + new = dev_alloc_skb(resp_len);
>> + if (!new) {
>> + printk(KERN_DEBUG "%s: failed to allocate buffer for probe "
>> + "response template\n", sdata->name);
>> + return -ENOMEM;
>
> I'd remove that message -- userspace already knows, and it's the only
> thing that really needs to?
>
Sure. I'll fix this in v3 (somehow missed this email before).
Arik
--
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
