Feature #9122
libstand: Consolidate tftp sendrecv into net.c sendrecv
Start date:
2018-02-16
Due date:
% Done:
100%
Estimated time:
Difficulty:
Medium
Tags:
Description
Update from FreeBSD: https://svnweb.freebsd.org/base?view=revision&revision=329339
bootp/arp/rarp/rpc all use the sendrecv implementation in net.c. tftp has
its own implementation because it passes an extra parameter into the recv
callback for the received payload type to be held.
These sendrecv implementations are otherwise equivalent, so consolidate
them. The other users of sendrecv won't be using the extra argument to recv,
but this gives us only one place to worry about respecting timeouts and one
consistent timeout behavior.
Tested by: sbruno
Reviewed by: sbruno, tsoome
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D14373
History
Updated by Electric Monk over 1 year ago
- % Done changed from 90 to 100
- Status changed from In Progress to Closed
git commit 890c86716e81b6b22639ff07791007b82c26e6c5
commit 890c86716e81b6b22639ff07791007b82c26e6c5 Author: Toomas Soome <tsoome@me.com> Date: 2018-05-14T17:00:24.000Z 9122 libstand: Consolidate tftp sendrecv into net.c sendrecv Reviewed by: Yuri Pankov <yuripv@yuripv.net> Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk> Approved by: Dan McDonald <danmcd@joyent.com>