From: Andre Renaud andre@designa-electronics.com
Avoid generating this section if there is nothing in it.
Signed-off-by: Andre Renaud andre@designa-electronics.com Signed-off-by: Simon Glass sjg@chromium.org Acked-by: Joe Hershberger joe.hershberger@ni.com ---
Changes in v2: None
net/bootp.c | 9 +++++++++ 1 file changed, 9 insertions(+)
diff --git a/net/bootp.c b/net/bootp.c index d7852db..71f09bd 100644 --- a/net/bootp.c +++ b/net/bootp.c @@ -655,6 +655,15 @@ static int bootp_extended(u8 *e)
*e++ = 255; /* End of the list */
+ /* + * If nothing in list, remove it altogether. Some DHCP servers get + * upset by this minor faux pas and do not respond at all. + */ + if (e == start + 3) { + printf("*** Warning: no DHCP options requested\n"); + e -= 3; + } + return e - start; } #endif