[PATCH] Omit P2P Group Info, in case of no connected peers

Arend van Spriel arend
Tue Mar 19 13:51:44 PDT 2013


On 03/19/2013 09:35 PM, Jan Ceuleers wrote:
> One review comment below.
>
> On 03/19/2013 08:26 AM, Chaitanya TK wrote:
>>
>>
>> As per P2P-sec V1.2: "The P2P Group
>> Info attribute shall be omitted if there are zero
>> connected P2P Clients."
>>
>> Don't add the IE, if the no of peers are zero.
>>
>> Signed-off-by: T Krushna Chaitanya <chaitanyatk at posedge.com>

Should be: Signed-hostap:

See CONTRIBUTIONS file.

Also the email address does not match the gmail address you used to send 
it. Not sure if that is acceptable.

>> ---
>>
>>   src/p2p/p2p_group.c |   16 +++++++++-------
>>   1 files changed, 9 insertions(+), 7 deletions(-)
>>
>> diff --git a/src/p2p/p2p_group.c b/src/p2p/p2p_group.c
>> index 9559e44..7adab90 100644
>> --- a/src/p2p/p2p_group.c
>> +++ b/src/p2p/p2p_group.c
>> @@ -413,14 +413,16 @@ static struct wpabuf * p2p_group_build_probe_resp_ie(struct p2p_group *group)
>>   	/* P2P Device Info */
>>   	p2p_buf_add_device_info(p2p_subelems, group->p2p, NULL);
>>
>> -	/* P2P Group Info */
>> -	group_info = wpabuf_put(p2p_subelems, 0);
>> -	wpabuf_put_u8(p2p_subelems, P2P_ATTR_GROUP_INFO);
>> -	wpabuf_put_le16(p2p_subelems, 0); /* Length to be filled */
>> -	for (m = group->members; m; m = m->next)
>> -		p2p_client_info(p2p_subelems, m);
>> -	WPA_PUT_LE16(group_info + 1,
>> +	/* P2P Group Info: Only when at least 1 P2P CLI is connected */
>> +	if (group->members != 0) {
>> +		group_info = wpabuf_put(p2p_subelems, 0);
>> +		wpabuf_put_u8(p2p_subelems, P2P_ATTR_GROUP_INFO);
>> +		wpabuf_put_le16(p2p_subelems, 0); /* Length to be filled */
>> +		for (m = group->members; m; m = m->next)
>> +			p2p_client_info(p2p_subelems, m);
>> +		WPA_PUT_LE16(group_info + 1,
>>   		     (u8 *) wpabuf_put(p2p_subelems, 0) - group_info - 3);
>
> Purely based on visual inspection I rather suspect that there should be
> a "+" at the beginning of the above line.
>
> How did you generate this patch?
>

Might be an email client issue.

Regards,
Arend




More information about the Hostap mailing list