From 27af5a0dd34b1b392aa69b548e18935aafe66e5f Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 3 Sep 2006 12:21:59 +0000 Subject: [PATCH] replacing func() with xfunc() where appropriate --- libbb/xfuncs.c | 2 +- networking/arping.c | 7 ++----- networking/fakeidentd.c | 2 +- networking/libiproute/iptunnel.c | 12 ++++++------ networking/udhcp/common.c | 5 +---- networking/zcip.c | 5 +---- 6 files changed, 12 insertions(+), 21 deletions(-) diff --git a/libbb/xfuncs.c b/libbb/xfuncs.c index b2b53a712..116cff459 100644 --- a/libbb/xfuncs.c +++ b/libbb/xfuncs.c @@ -454,7 +454,7 @@ DIR *xopendir(const char *path) /* Die with an error message if we can't daemonize. */ void xdaemon(int nochdir, int noclose) { - if (daemon(nochdir, noclose)) bb_perror_msg_and_die("daemon"); + if (daemon(nochdir, noclose)) bb_perror_msg_and_die("daemon"); } #endif #endif diff --git a/networking/arping.c b/networking/arping.c index 5665ddb2b..08b27273f 100644 --- a/networking/arping.c +++ b/networking/arping.c @@ -259,7 +259,7 @@ int arping_main(int argc, char **argv) char *source = NULL; char *target; - s = socket(PF_PACKET, SOCK_DGRAM, 0); + s = xsocket(PF_PACKET, SOCK_DGRAM, 0); ifindex = errno; // Drop suid root privileges @@ -346,11 +346,8 @@ int arping_main(int argc, char **argv) if (!(cfg&dad) || src.s_addr) { struct sockaddr_in saddr; - int probe_fd = socket(AF_INET, SOCK_DGRAM, 0); /* maybe use bb_xsocket? */ + int probe_fd = xsocket(AF_INET, SOCK_DGRAM, 0); - if (probe_fd < 0) { - bb_error_msg_and_die("socket"); - } if (device) { if (setsockopt (probe_fd, SOL_SOCKET, SO_BINDTODEVICE, device, diff --git a/networking/fakeidentd.c b/networking/fakeidentd.c index b965147e1..29e09d13e 100644 --- a/networking/fakeidentd.c +++ b/networking/fakeidentd.c @@ -135,7 +135,7 @@ static int godaemon(void) switch (fork()) { case -1: - bb_perror_msg_and_die("Could not fork"); + bb_perror_msg_and_die("fork"); case 0: pw = getpwnam(nobodystr); diff --git a/networking/libiproute/iptunnel.c b/networking/libiproute/iptunnel.c index 4524d4954..806d8657d 100644 --- a/networking/libiproute/iptunnel.c +++ b/networking/libiproute/iptunnel.c @@ -43,7 +43,7 @@ static int do_ioctl_get_ifindex(char *dev) int fd; strcpy(ifr.ifr_name, dev); - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); if (ioctl(fd, SIOCGIFINDEX, &ifr)) { bb_perror_msg("ioctl"); return 0; @@ -58,7 +58,7 @@ static int do_ioctl_get_iftype(char *dev) int fd; strcpy(ifr.ifr_name, dev); - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); if (ioctl(fd, SIOCGIFHWADDR, &ifr)) { bb_perror_msg("ioctl"); return -1; @@ -74,7 +74,7 @@ static char *do_ioctl_get_ifname(int idx) int fd; ifr.ifr_ifindex = idx; - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); if (ioctl(fd, SIOCGIFNAME, &ifr)) { bb_perror_msg("ioctl"); return NULL; @@ -93,7 +93,7 @@ static int do_get_ioctl(char *basedev, struct ip_tunnel_parm *p) strcpy(ifr.ifr_name, basedev); ifr.ifr_ifru.ifru_data = (void*)p; - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); err = ioctl(fd, SIOCGETTUNNEL, &ifr); if (err) { bb_perror_msg("ioctl"); @@ -114,7 +114,7 @@ static int do_add_ioctl(int cmd, char *basedev, struct ip_tunnel_parm *p) strcpy(ifr.ifr_name, basedev); } ifr.ifr_ifru.ifru_data = (void*)p; - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); err = ioctl(fd, cmd, &ifr); if (err) { bb_perror_msg("ioctl"); @@ -135,7 +135,7 @@ static int do_del_ioctl(char *basedev, struct ip_tunnel_parm *p) strcpy(ifr.ifr_name, basedev); } ifr.ifr_ifru.ifru_data = (void*)p; - fd = socket(AF_INET, SOCK_DGRAM, 0); + fd = xsocket(AF_INET, SOCK_DGRAM, 0); err = ioctl(fd, SIOCDELTUNNEL, &ifr); if (err) { bb_perror_msg("ioctl"); diff --git a/networking/udhcp/common.c b/networking/udhcp/common.c index 2c902fc01..c2025e588 100644 --- a/networking/udhcp/common.c +++ b/networking/udhcp/common.c @@ -59,10 +59,7 @@ void udhcp_background(const char *pidfile) /* hold lock during fork. */ pid_fd = pidfile_acquire(pidfile); - if (daemon(0, 0) == -1) { /* bb_xdaemon? */ - perror("fork"); - exit(1); - } + xdaemon(0, 0); daemonized++; pidfile_write_release(pid_fd); #endif /* __uClinux__ */ diff --git a/networking/zcip.c b/networking/zcip.c index 8e6d8792d..e9dd05011 100644 --- a/networking/zcip.c +++ b/networking/zcip.c @@ -288,10 +288,7 @@ fail: // daemonize now; don't delay system startup if (!foreground) { - if (daemon(0, verbose) < 0) { - why = "daemon"; - goto bad; - } + xdaemon(0, verbose); syslog(LOG_INFO, "start, interface %s", intf); }