diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-02-25 23:23:58 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-02-25 23:23:58 +0000 |
| commit | 6f1713f216fef686a68db5ee02232bc67e525c7d (patch) | |
| tree | 256adc75a88723a4b247c32e692d721d65243a82 /libbb | |
| parent | 394eebed6656dfc2e56a79500b602023000ac415 (diff) | |
| download | busybox-w32-6f1713f216fef686a68db5ee02232bc67e525c7d.tar.gz busybox-w32-6f1713f216fef686a68db5ee02232bc67e525c7d.tar.bz2 busybox-w32-6f1713f216fef686a68db5ee02232bc67e525c7d.zip | |
*: intrduce and use safe_gethostname. By Tito <farmatito AT tiscali.it>
safe_gethostname - 48 +48
glob3 35 37 +2
timestamp_and_log 314 315 +1
udhcp_send_kernel_packet 234 231 -3
scan_tree 275 271 -4
passwd_main 1074 1070 -4
print_login_prompt 68 58 -10
obscure 392 377 -15
syslogd_main 882 866 -16
print_login_issue 516 478 -38
hostname_main 278 223 -55
parse_and_put_prompt 825 756 -69
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 2/9 up/down: 51/-214) Total: -163 bytes
text data bss dec hex filename
798791 728 7484 807003 c505b busybox_old
798631 728 7484 806843 c4fbb busybox_unstripped
Diffstat (limited to 'libbb')
| -rw-r--r-- | libbb/Kbuild | 1 | ||||
| -rw-r--r-- | libbb/lineedit.c | 6 | ||||
| -rw-r--r-- | libbb/login.c | 14 | ||||
| -rw-r--r-- | libbb/obscure.c | 13 |
4 files changed, 13 insertions, 21 deletions
diff --git a/libbb/Kbuild b/libbb/Kbuild index 515368de7..aab016e85 100644 --- a/libbb/Kbuild +++ b/libbb/Kbuild | |||
| @@ -73,6 +73,7 @@ lib-y += recursive_action.o | |||
| 73 | lib-y += remove_file.o | 73 | lib-y += remove_file.o |
| 74 | lib-y += restricted_shell.o | 74 | lib-y += restricted_shell.o |
| 75 | lib-y += run_shell.o | 75 | lib-y += run_shell.o |
| 76 | lib-y += safe_gethostname.o | ||
| 76 | lib-y += safe_poll.o | 77 | lib-y += safe_poll.o |
| 77 | lib-y += safe_strncpy.o | 78 | lib-y += safe_strncpy.o |
| 78 | lib-y += safe_write.o | 79 | lib-y += safe_write.o |
diff --git a/libbb/lineedit.c b/libbb/lineedit.c index 9aab63702..c6aa45c93 100644 --- a/libbb/lineedit.c +++ b/libbb/lineedit.c | |||
| @@ -1203,11 +1203,7 @@ static void parse_and_put_prompt(const char *prmt_ptr) | |||
| 1203 | break; | 1203 | break; |
| 1204 | #endif | 1204 | #endif |
| 1205 | case 'h': | 1205 | case 'h': |
| 1206 | pbuf = free_me = xzalloc(256); | 1206 | pbuf = free_me = safe_gethostname(); |
| 1207 | if (gethostname(pbuf, 255) < 0) { | ||
| 1208 | pbuf[0] = '?'; | ||
| 1209 | pbuf[1] = '\0'; | ||
| 1210 | } | ||
| 1211 | *strchrnul(pbuf, '.') = '\0'; | 1207 | *strchrnul(pbuf, '.') = '\0'; |
| 1212 | break; | 1208 | break; |
| 1213 | case '$': | 1209 | case '$': |
diff --git a/libbb/login.c b/libbb/login.c index d1f5d6498..a711a5437 100644 --- a/libbb/login.c +++ b/libbb/login.c | |||
| @@ -50,6 +50,7 @@ void print_login_issue(const char *issue_file, const char *tty) | |||
| 50 | outbuf = uts.sysname; | 50 | outbuf = uts.sysname; |
| 51 | break; | 51 | break; |
| 52 | case 'n': | 52 | case 'n': |
| 53 | case 'h': | ||
| 53 | outbuf = uts.nodename; | 54 | outbuf = uts.nodename; |
| 54 | break; | 55 | break; |
| 55 | case 'r': | 56 | case 'r': |
| @@ -72,10 +73,6 @@ void print_login_issue(const char *issue_file, const char *tty) | |||
| 72 | case 't': | 73 | case 't': |
| 73 | strftime(buf, sizeof(buf), fmtstr_t, localtime(&t)); | 74 | strftime(buf, sizeof(buf), fmtstr_t, localtime(&t)); |
| 74 | break; | 75 | break; |
| 75 | case 'h': | ||
| 76 | gethostname(buf, sizeof(buf) - 1); | ||
| 77 | buf[sizeof(buf) - 1] = '\0'; | ||
| 78 | break; | ||
| 79 | case 'l': | 76 | case 'l': |
| 80 | outbuf = tty; | 77 | outbuf = tty; |
| 81 | break; | 78 | break; |
| @@ -91,13 +88,12 @@ void print_login_issue(const char *issue_file, const char *tty) | |||
| 91 | 88 | ||
| 92 | void print_login_prompt(void) | 89 | void print_login_prompt(void) |
| 93 | { | 90 | { |
| 94 | char buf[MAXHOSTNAMELEN+1]; | 91 | char *hostname = safe_gethostname(); |
| 95 | 92 | ||
| 96 | if (gethostname(buf, MAXHOSTNAMELEN) == 0) | 93 | fputs(hostname, stdout); |
| 97 | fputs(buf, stdout); | ||
| 98 | |||
| 99 | fputs(LOGIN, stdout); | 94 | fputs(LOGIN, stdout); |
| 100 | fflush(stdout); | 95 | fflush(stdout); |
| 96 | free(hostname); | ||
| 101 | } | 97 | } |
| 102 | 98 | ||
| 103 | /* Clear dangerous stuff, set PATH */ | 99 | /* Clear dangerous stuff, set PATH */ |
diff --git a/libbb/obscure.c b/libbb/obscure.c index 5cc906235..1841b27d6 100644 --- a/libbb/obscure.c +++ b/libbb/obscure.c | |||
| @@ -93,7 +93,7 @@ static const char *obscure_msg(const char *old_p, const char *new_p, const struc | |||
| 93 | /* Add 2 for each type of characters to the minlen of password */ | 93 | /* Add 2 for each type of characters to the minlen of password */ |
| 94 | int size = CONFIG_PASSWORD_MINLEN + 8; | 94 | int size = CONFIG_PASSWORD_MINLEN + 8; |
| 95 | const char *p; | 95 | const char *p; |
| 96 | char hostname[255]; | 96 | char *hostname; |
| 97 | 97 | ||
| 98 | /* size */ | 98 | /* size */ |
| 99 | if (!new_p || (length = strlen(new_p)) < CONFIG_PASSWORD_MINLEN) | 99 | if (!new_p || (length = strlen(new_p)) < CONFIG_PASSWORD_MINLEN) |
| @@ -108,12 +108,11 @@ static const char *obscure_msg(const char *old_p, const char *new_p, const struc | |||
| 108 | return "similar to gecos"; | 108 | return "similar to gecos"; |
| 109 | } | 109 | } |
| 110 | /* hostname as-is, as sub-string, reversed, capitalized, doubled */ | 110 | /* hostname as-is, as sub-string, reversed, capitalized, doubled */ |
| 111 | if (gethostname(hostname, 255) == 0) { | 111 | hostname = safe_gethostname(); |
| 112 | hostname[254] = '\0'; | 112 | i = string_checker(new_p, hostname); |
| 113 | if (string_checker(new_p, hostname)) { | 113 | free(hostname); |
| 114 | return "similar to hostname"; | 114 | if (i) |
| 115 | } | 115 | return "similar to hostname"; |
| 116 | } | ||
| 117 | 116 | ||
| 118 | /* Should / Must contain a mix of: */ | 117 | /* Should / Must contain a mix of: */ |
| 119 | for (i = 0; i < length; i++) { | 118 | for (i = 0; i < length; i++) { |
