Re: [PATCH] DCCP: handle EAGAIN case

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

 



Quoting Ian McDonald:
|  >  a) You helped to uncover at least one real bug here:
|  >
|  >     long delay;
|  >
|  >     delay = msecs_to_jiffies(rc);
|  >     if (delay > *timeo || delay < 0)
|  >
|  
|  I was thinking of changing the type from long to unsigned long to
|  match and removing the test too. I agree totally.
For this case I have prepared a patch. With regard to the EAGAIN case I was
half waiting for Arnaldo to say something, for the moment I have put it here:
http://www.erg.abdn.ac.uk/users/gerrit/dccp/experimental_patches/15a_handle_EAGAIN.diff

I think we should fix the EAGAIN case. Maybe also something for EINTR (err=-4).
-
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