Re: [PATCH 8/8]: Support inserting options during the 3-way handshake

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Arnaldo Carvalho de Melo:
|  > --- a/net/dccp/options.c
|  > +++ b/net/dccp/options.c
|  > @@ -160,7 +160,7 @@ int dccp_parse_options(struct sock *sk, 
|  >                       if (len != 4)
|  >                               goto out_invalid_option;
|  >  
|  > -                     tse = dreq? &dreq->dreq_tstamp : dp->dccps_tstamp;
|  > +                     tse = dreq? &dreq->dreq_tstamp : &dp->dccps_tstamp;
|  
|  And here you fix it in an unrelated patch, tsc, tsc :-)
These kind of errors don't happen on purpose, if I submitted everything as one big patch 
no one would notice. But with one problem split into several tens of patches it easily 
happens that one hunk is fixed while quilt is in a different patch - and then it gets assigned 
the wrong patch. I missed this hunk so thank you two for spotting this - it is fixed now.
-
To unsubscribe from this list: send the line "unsubscribe dccp" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [IETF DCCP]     [Linux Networking]     [Git]     [Security]     [Linux Assembly]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]

  Powered by Linux