Patch for review

Rahul Jain rahul.jain at samsung.com
Fri Feb 21 03:58:04 EST 2014


hello Jouni,

 As requested, I am attaching patch with signed off.
==
rom 2d330d4b823a8aee9a509af4ea9a696baa8fad57 Mon Sep 17 00:00:00 2001
From: Rahul Jain <rahul.jain at samsung.com>
Date: Thu, 20 Feb 2014 16:34:09 +0530
Subject: [PATCH]  Fix for missing eloop_cancel_timeout  when p2p invitation
 is received


Signed-off-by: Rahul Jain <rahul.jain at samsung.com>
---
 src/p2p/p2p.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/src/p2p/p2p.c b/src/p2p/p2p.c
index 2ecc547..1e9699e 100644
--- a/src/p2p/p2p.c
+++ b/src/p2p/p2p.c
@@ -2145,6 +2145,7 @@ p2p_probe_req_rx(struct p2p_data *p2p, const u8 *addr, const u8 *dst,
      == 0) {
   /* Received a Probe Request from Invite peer */
   p2p_dbg(p2p, "Found Invite peer - try to start Invite from timeout");
+  eloop_cancel_timeout(p2p_invite_start, p2p, NULL);
   eloop_register_timeout(0, 0, p2p_invite_start, p2p, NULL);
   return P2P_PREQ_PROCESSED;
  }
-- 
1.7.9.5
==

Thanks
-Rahul



------- Original Message -------
Sender : Jouni Malinen<j at w1.fi>
Date : Feb 21, 2014 14:12 (GMT+05:30)
Title : Re: Patch for review

On Fri, Feb 21, 2014 at 03:40:01AM +0000, Rahul Jain wrote:
> [PATCH]  Fix for missing eloop_cancel_timeout  when p2p invitation
 is received

> 
diff --git a/src/p2p/p2p.c b/src/p2p/p2p.c
index 2ecc547..1e9699e 100644
--- a/src/p2p/p2p.c
+++ b/src/p2p/p2p.c
@@ -2145,6 +2145,7 @@ p2p_probe_req_rx(struct p2p_data *p2p, const u8 *addr, const u8 *dst,
      == 0) {
   /* Received a Probe Request from Invite peer */
   p2p_dbg(p2p, "Found Invite peer - try to start Invite from timeout");
+  eloop_cancel_timeout(p2p_invite_start, p2p, NULL);
   eloop_register_timeout(0, 0, p2p_invite_start, p2p, NULL);
   return P2P_PREQ_PROCESSED;
  }


The actual change itself looks reasonable, but I would need to receive a
patch with Signed-off-by line in the commit message to be able to apply
this. Could you please read the CONTRIBUTIONS file(*) and re-send this
with the Signed-off-by added as described in that file?

Please also note that HTML-only email is discouraged on this mailing
list and it would be highly preferred to get a plaintext version. As you
can see from the quoted version of the change above, this didn't look
very clear in my email reader... :)


(*) http://w1.fi/cgit/hostap/tree/CONTRIBUTIONS

-- 
Jouni Malinen                                            PGP id EFC895FA



Thanks
-Rahul Jain
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-for-missing-eloop_cancel_timeout-when-p2p-invita.patch
Type: application/octet-stream
Size: 860 bytes
Desc: not available
URL: <http://lists.shmoo.com/pipermail/hostap/attachments/20140221/9eeb2566/attachment-0001.obj>


More information about the HostAP mailing list