[PATCH 2/5] p2p: update dialog_token on failures as well

Jouni Malinen j
Sun Mar 22 10:11:21 PDT 2015


On Sun, Mar 22, 2015 at 11:58:26AM +0200, Eliad Peller wrote:
> On Fri, Mar 20, 2015 at 3:18 PM, Jouni Malinen <j at w1.fi> wrote:
> > Could you please clarify what this solves and which frame would be the
> > target for preventing the same dialog token from being used again? The
> > change here is in p2p_process_go_neg_req() and that function does not
> > verify dialog_token matching (only GO Neg Resp and Conf has such code as
> > far as GO Negotiation is concerned).
> >
> this doesn't solve any specific scenario, but just seems like the
> right thing to do in order to prevent the same dialog token from being
> reused (e.g. when sending a new provision discovery).

Provision Discovery? Why would take make any difference? The dialog
token is specific to each request/response(/confirm) transaction for a
specific type of frames; it does not have any meaning between GO
Negotiation frames and Provision Discovery frames. If fetching a dialog
token from a GO Negotiation frame and then using that dialog token in a
Provision Discovery frame is the intent here, I don't see why that would
be of any benefit.

-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list