diff options
| author | Denys Vlasenko | 2010-03-19 23:42:23 +0100 |
|---|---|---|
| committer | Denys Vlasenko | 2010-03-19 23:42:23 +0100 |
| commit | 777706cb2352c02602842f89d916a0e18ce6bd91 (patch) | |
| tree | 75596b337731c87718c9015eb9eb6730183824dd /networking/udhcp/script.c | |
| parent | 77a2c51e79eb54c8f5cc8903465223cbac6e8d50 (diff) | |
| download | busybox-777706cb2352c02602842f89d916a0e18ce6bd91.zip busybox-777706cb2352c02602842f89d916a0e18ce6bd91.tar.gz | |
udhcp: remove support for some really old and odd options
function old new delta
dhcp_options 72 68 -4
dhcp_option_strings 271 253 -18
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'networking/udhcp/script.c')
| -rw-r--r-- | networking/udhcp/script.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/networking/udhcp/script.c b/networking/udhcp/script.c index a74697c..dc658ad 100644 --- a/networking/udhcp/script.c +++ b/networking/udhcp/script.c @@ -78,8 +78,11 @@ static NOINLINE char *xmalloc_optname_optval(uint8_t *option, const struct dhcp_ *dest++ = '/'; option += 4; optlen = 4; - case OPTION_IP: /* Works regardless of host byte order. */ + case OPTION_IP: dest += sprint_nip(dest, "", option); +// TODO: it can be a list only if (type_p->flags & OPTION_LIST). +// Should we bail out/warn if we see multi-ip option which is +// not allowed to be such? For example, DHCP_BROADCAST... break; case OPTION_BOOLEAN: dest += sprintf(dest, *option ? "yes" : "no"); |