From 6a52d0d155fc8b892e30cdb5e3fbe4c2779bd4cd Mon Sep 17 00:00:00 2001 From: ajs Date: Sun, 30 Jan 2005 18:49:28 +0000 Subject: 2005-01-30 Andrew J. Schorr * daemon.c: (daemon) Replace perror with zlog_err. * vty.c: (vty_serv_un) Replace perror with zlog_err. * ripd.c: (rip_create_socket) Replace perror with zlog_err. * ioctl.c: (if_ioctl,if_ioctl_ipv6,if_get_flags) Replace perror with zlog_err. * ioctl_solaris.c: (if_ioctl,if_ioctl_ipv6) Replace perror with zlog_err. --- lib/ChangeLog | 5 +++++ lib/daemon.c | 4 ++-- lib/vty.c | 6 +++--- ripd/ChangeLog | 4 ++++ ripd/ripd.c | 2 +- zebra/ChangeLog | 7 +++++++ zebra/ioctl.c | 9 ++++++--- zebra/ioctl_solaris.c | 7 +++++-- 8 files changed, 33 insertions(+), 11 deletions(-) diff --git a/lib/ChangeLog b/lib/ChangeLog index 34dad166..dcf37767 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,3 +1,8 @@ +2005-01-30 Andrew J. Schorr + + * daemon.c: (daemon) Replace perror with zlog_err. + * vty.c: (vty_serv_un) Replace perror with zlog_err. + 2005-01-29 Andrew J. Schorr * buffer.h: Fix comment on buffer_getstr to reflect that it now diff --git a/lib/daemon.c b/lib/daemon.c index d8380106..e73a74f1 100644 --- a/lib/daemon.c +++ b/lib/daemon.c @@ -35,7 +35,7 @@ daemon (int nochdir, int noclose) /* In case of fork is error. */ if (pid < 0) { - perror ("fork"); + zlog_err ("fork failed: %s", safe_strerror(errno)); return -1; } @@ -48,7 +48,7 @@ daemon (int nochdir, int noclose) if (pid == -1) { - perror ("setsid"); + zlog_err ("setsid failed: %s", safe_strerror(errno)); return -1; } diff --git a/lib/vty.c b/lib/vty.c index 42487330..053e9ee8 100644 --- a/lib/vty.c +++ b/lib/vty.c @@ -1824,7 +1824,7 @@ vty_serv_un (const char *path) sock = socket (AF_UNIX, SOCK_STREAM, 0); if (sock < 0) { - perror ("sock"); + zlog_err("Cannot create unix stream socket: %s", safe_strerror(errno)); return; } @@ -1841,7 +1841,7 @@ vty_serv_un (const char *path) ret = bind (sock, (struct sockaddr *) &serv, len); if (ret < 0) { - perror ("bind"); + zlog_err("Cannot bind path %s: %s", path, safe_strerror(errno)); close (sock); /* Avoid sd leak. */ return; } @@ -1849,7 +1849,7 @@ vty_serv_un (const char *path) ret = listen (sock, 5); if (ret < 0) { - perror ("listen"); + zlog_err("listen(fd %d) failed: %s", sock, safe_strerror(errno)); close (sock); /* Avoid sd leak. */ return; } diff --git a/ripd/ChangeLog b/ripd/ChangeLog index 5a5d1b4b..60be27ea 100644 --- a/ripd/ChangeLog +++ b/ripd/ChangeLog @@ -1,3 +1,7 @@ +2005-01-30 Andrew J. Schorr + + * ripd.c: (rip_create_socket) Replace perror with zlog_err. + 2005-01-30 Andrew J. Schorr * ripd.c: (rip_create_socket) Save errno before calling diff --git a/ripd/ripd.c b/ripd/ripd.c index 2a345a66..b5d130e8 100644 --- a/ripd/ripd.c +++ b/ripd/ripd.c @@ -1985,7 +1985,7 @@ rip_create_socket () sock = socket (AF_INET, SOCK_DGRAM, 0); if (sock < 0) { - perror ("socket"); + zlog_err("Cannot create UDP socket: %s", safe_strerror(errno)); exit (1); } diff --git a/zebra/ChangeLog b/zebra/ChangeLog index 0ebe7052..1f97fef9 100644 --- a/zebra/ChangeLog +++ b/zebra/ChangeLog @@ -1,3 +1,10 @@ +2005-01-30 Andrew J. Schorr + + * ioctl.c: (if_ioctl,if_ioctl_ipv6,if_get_flags) Replace perror with + zlog_err. + * ioctl_solaris.c: (if_ioctl,if_ioctl_ipv6) Replace perror with + zlog_err. + 2005-01-29 Andrew J. Schorr * if_ioctl_solaris.c: (interface_list_ioctl) Save errno before calling diff --git a/zebra/ioctl.c b/zebra/ioctl.c index e28d359e..56de810c 100644 --- a/zebra/ioctl.c +++ b/zebra/ioctl.c @@ -54,9 +54,10 @@ if_ioctl (u_long request, caddr_t buffer) sock = socket (AF_INET, SOCK_DGRAM, 0); if (sock < 0) { + int save_errno = errno; if (zserv_privs.change(ZPRIVS_LOWER)) zlog (NULL, LOG_ERR, "Can't lower privileges"); - perror ("socket"); + zlog_err("Cannot create UDP socket: %s", safe_strerror(save_errno)); exit (1); } if ((ret = ioctl (sock, request, buffer)) < 0) @@ -86,9 +87,11 @@ if_ioctl_ipv6 (u_long request, caddr_t buffer) sock = socket (AF_INET6, SOCK_DGRAM, 0); if (sock < 0) { + int save_errno = errno; if (zserv_privs.change(ZPRIVS_LOWER)) zlog (NULL, LOG_ERR, "Can't lower privileges"); - perror ("socket"); + zlog_err("Cannot create IPv6 datagram socket: %s", + safe_strerror(save_errno)); exit (1); } @@ -343,7 +346,7 @@ if_get_flags (struct interface *ifp) ret = if_ioctl (SIOCGIFFLAGS, (caddr_t) &ifreq); if (ret < 0) { - perror ("ioctl"); + zlog_err("if_ioctl(SIOCGIFFLAGS) failed: %s", safe_strerror(errno)); return; } diff --git a/zebra/ioctl_solaris.c b/zebra/ioctl_solaris.c index 5ec327ac..43e1e83f 100644 --- a/zebra/ioctl_solaris.c +++ b/zebra/ioctl_solaris.c @@ -55,9 +55,10 @@ if_ioctl (u_long request, caddr_t buffer) sock = socket (AF_INET, SOCK_DGRAM, 0); if (sock < 0) { + int save_errno = errno; if (zserv_privs.change(ZPRIVS_LOWER)) zlog (NULL, LOG_ERR, "Can't lower privileges"); - perror ("socket"); + zlog_err("Cannot create UDP socket: %s", safe_strerror(save_errno)); exit (1); } @@ -92,9 +93,11 @@ if_ioctl_ipv6 (u_long request, caddr_t buffer) sock = socket (AF_INET6, SOCK_DGRAM, 0); if (sock < 0) { + int save_errno = errno; if (zserv_privs.change(ZPRIVS_LOWER)) zlog (NULL, LOG_ERR, "Can't lower privileges"); - perror ("socket"); + zlog_err("Cannot create IPv6 datagram socket: %s", + safe_strerror(save_errno)); exit (1); } -- cgit v1.2.1