aboutsummaryrefslogtreecommitdiff
path: root/libbb
diff options
context:
space:
mode:
authorDenis Vlasenko <vda.linux@googlemail.com>2007-09-27 10:09:59 +0000
committerDenis Vlasenko <vda.linux@googlemail.com>2007-09-27 10:09:59 +0000
commit5d61e71c3a8ac3296afbfe9a014c62050c5a9234 (patch)
treee9ab2cd35d1f2d51b26a678551a0439b1719928b /libbb
parentc9dc2ac578278e86be248cc21e53081f7054da83 (diff)
downloadbusybox-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/Kbuild1
-rw-r--r--libbb/safe_poll.c34
-rw-r--r--libbb/safe_strncpy.c2
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
71lib-y += remove_file.o 71lib-y += remove_file.o
72lib-y += restricted_shell.o 72lib-y += restricted_shell.o
73lib-y += run_shell.o 73lib-y += run_shell.o
74lib-y += safe_poll.o
74lib-y += safe_strncpy.o 75lib-y += safe_strncpy.o
75lib-y += safe_write.o 76lib-y += safe_write.o
76lib-y += setup_environment.o 77lib-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! */
15int 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. */
13char * safe_strncpy(char *dst, const char *src, size_t size) 13char *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';