diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-09-27 10:09:59 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-09-27 10:09:59 +0000 |
commit | 5d61e71c3a8ac3296afbfe9a014c62050c5a9234 (patch) | |
tree | e9ab2cd35d1f2d51b26a678551a0439b1719928b /libbb | |
parent | c9dc2ac578278e86be248cc21e53081f7054da83 (diff) | |
download | busybox-w32-5d61e71c3a8ac3296afbfe9a014c62050c5a9234.tar.gz busybox-w32-5d61e71c3a8ac3296afbfe9a014c62050c5a9234.tar.bz2 busybox-w32-5d61e71c3a8ac3296afbfe9a014c62050c5a9234.zip |
introduce safe_poll (fixes a problem in top)
function old new delta
safe_poll - 77 +77
svlogd_main 1470 1466 -4
zcip_main 1530 1524 -6
forkexec 1345 1338 -7
decode_format_string 795 788 -7
collect_blk 474 467 -7
buffer_pread 540 532 -8
tftp 1182 1172 -10
microcom_main 763 749 -14
arpping 441 424 -17
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 0/9 up/down: 77/-80) Total: -3 bytes
text data bss dec hex filename
770162 1034 10404 781600 bed20 busybox_old
770158 1034 10404 781596 bed1c busybox_unstripped
Diffstat (limited to 'libbb')
-rw-r--r-- | libbb/Kbuild | 1 | ||||
-rw-r--r-- | libbb/safe_poll.c | 34 | ||||
-rw-r--r-- | libbb/safe_strncpy.c | 2 |
3 files changed, 36 insertions, 1 deletions
diff --git a/libbb/Kbuild b/libbb/Kbuild index 1800bde8f..55b3658b4 100644 --- a/libbb/Kbuild +++ b/libbb/Kbuild | |||
@@ -71,6 +71,7 @@ lib-y += recursive_action.o | |||
71 | lib-y += remove_file.o | 71 | lib-y += remove_file.o |
72 | lib-y += restricted_shell.o | 72 | lib-y += restricted_shell.o |
73 | lib-y += run_shell.o | 73 | lib-y += run_shell.o |
74 | lib-y += safe_poll.o | ||
74 | lib-y += safe_strncpy.o | 75 | lib-y += safe_strncpy.o |
75 | lib-y += safe_write.o | 76 | lib-y += safe_write.o |
76 | lib-y += setup_environment.o | 77 | lib-y += setup_environment.o |
diff --git a/libbb/safe_poll.c b/libbb/safe_poll.c new file mode 100644 index 000000000..d2b773c36 --- /dev/null +++ b/libbb/safe_poll.c | |||
@@ -0,0 +1,34 @@ | |||
1 | /* vi: set sw=4 ts=4: */ | ||
2 | /* | ||
3 | * Utility routines. | ||
4 | * | ||
5 | * Copyright (C) 2007 by Denys Vlasenko <vda.linux@googlemail.com> | ||
6 | * | ||
7 | * Licensed under GPLv2, see file LICENSE in this tarball for details. | ||
8 | */ | ||
9 | |||
10 | #include "libbb.h" | ||
11 | |||
12 | /* Wrapper which restarts poll on EINTR or ENOMEM. | ||
13 | * On other errors does perror("poll") and returns. | ||
14 | * Warning! May take longer than timeout_ms to return! */ | ||
15 | int safe_poll(struct pollfd *ufds, nfds_t nfds, int timeout) | ||
16 | { | ||
17 | while (1) { | ||
18 | int n = poll(ufds, nfds, timeout); | ||
19 | if (n >= 0) | ||
20 | return n; | ||
21 | /* Make sure we inch towards completion */ | ||
22 | if (timeout > 0) | ||
23 | timeout--; | ||
24 | /* E.g. strace causes poll to return this */ | ||
25 | if (errno == EINTR) | ||
26 | continue; | ||
27 | /* Kernel is very low on memory. Retry. */ | ||
28 | /* I doubt many callers would handle this correctly! */ | ||
29 | if (errno == ENOMEM) | ||
30 | continue; | ||
31 | bb_perror_msg("poll"); | ||
32 | return n; | ||
33 | } | ||
34 | } | ||
diff --git a/libbb/safe_strncpy.c b/libbb/safe_strncpy.c index ebc7e28f3..cc425839f 100644 --- a/libbb/safe_strncpy.c +++ b/libbb/safe_strncpy.c | |||
@@ -10,7 +10,7 @@ | |||
10 | #include "libbb.h" | 10 | #include "libbb.h" |
11 | 11 | ||
12 | /* Like strncpy but make sure the resulting string is always 0 terminated. */ | 12 | /* Like strncpy but make sure the resulting string is always 0 terminated. */ |
13 | char * safe_strncpy(char *dst, const char *src, size_t size) | 13 | char *safe_strncpy(char *dst, const char *src, size_t size) |
14 | { | 14 | { |
15 | if (!size) return dst; | 15 | if (!size) return dst; |
16 | dst[--size] = '\0'; | 16 | dst[--size] = '\0'; |