diff options
author | Ron Yorston <rmy@pobox.com> | 2015-10-12 10:51:25 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2015-10-13 14:56:22 +0200 |
commit | 4d0c1ea4784c9844f8468d97ca5c26d3c70f9921 (patch) | |
tree | 3130f0d48e0e370e20dc04ff270d0ed8e8388f6e | |
parent | ab3964db4e75e34f6f9347406c5fd2bced04f2dd (diff) | |
download | busybox-4d0c1ea4784c9844f8468d97ca5c26d3c70f9921.tar.gz |
wget: shrink code if https isn't supported
If FEATURE_WGET_OPENSSL and FEATURE_WGET_SSL_HELPER are both disabled
there's no point in checking for https URLs.
function old new delta
P_HTTPS 6 - -6
.rodata 155501 155469 -32
parse_url 476 423 -53
------------------------------------------------------------------------------
(add/remove: 0/1 grow/shrink: 0/2 up/down: 0/-91) Total: -91 bytes
Signed-off-by: Ron Yorston <rmy@pobox.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | networking/wget.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/networking/wget.c b/networking/wget.c index d4a9c0cb1..af9c53c22 100644 --- a/networking/wget.c +++ b/networking/wget.c @@ -147,7 +147,9 @@ struct host_info { }; static const char P_FTP[] = "ftp"; static const char P_HTTP[] = "http"; +#if ENABLE_FEATURE_WGET_OPENSSL || ENABLE_FEATURE_WGET_SSL_HELPER static const char P_HTTPS[] = "https"; +#endif #if ENABLE_FEATURE_WGET_LONG_OPTIONS /* User-specified headers prevent using our corresponding built-in headers. */ @@ -410,10 +412,12 @@ static void parse_url(const char *src_url, struct host_info *h) if (strcmp(url, P_FTP) == 0) { h->port = bb_lookup_port(P_FTP, "tcp", 21); } else +#if ENABLE_FEATURE_WGET_OPENSSL || ENABLE_FEATURE_WGET_SSL_HELPER if (strcmp(url, P_HTTPS) == 0) { h->port = bb_lookup_port(P_HTTPS, "tcp", 443); h->protocol = P_HTTPS; } else +#endif if (strcmp(url, P_HTTP) == 0) { http: h->port = bb_lookup_port(P_HTTP, "tcp", 80); |