hush: fix potential buffer overflow on NOMMU

[oweals/busybox.git] / networking / isrv_identd.c

diff --git a/networking/isrv_identd.c b/networking/isrv_identd.c
index 33338572d986e094981a242e7cec39a9e5f2c68c..e8ba007667255e51519062dc208a60355f551be5 100644 (file)
--- a/networking/isrv_identd.c
+++ b/networking/isrv_identd.c
@@ -61,7 +61,7 @@ static int do_rd(int fd, void **paramp)
        p = strpbrk(cur, "\r\n");
        if (p)
                *p = '\0';
-       if (!p && sz && buf->pos <= sizeof(buf->buf))
+       if (!p && sz && buf->pos <= (int)sizeof(buf->buf))
                goto ok;
        /* Terminate session. If we are in server mode, then
         * fd is still in nonblocking mode - we never block here */
@@ -76,7 +76,7 @@ static int do_rd(int fd, void **paramp)
        return retval;
 }
 
-static int do_timeout(void **paramp)
+static int do_timeout(void **paramp UNUSED_PARAM)
 {
        return 1; /* terminate session */
 }
@@ -93,7 +93,7 @@ static void inetd_mode(void)
 }
 
 int fakeidentd_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
-int fakeidentd_main(int argc, char **argv)
+int fakeidentd_main(int argc UNUSED_PARAM, char **argv)
 {
        enum {
                OPT_foreground = 0x1,
@@ -113,7 +113,7 @@ int fakeidentd_main(int argc, char **argv)
                strncpy(bogouser, argv[optind], sizeof(bogouser));
 
        /* Daemonize if no -f and no -i and no -w */
-       if (!(opt & OPT_fiw));
+       if (!(opt & OPT_fiw))
                bb_daemonize_or_rexec(0, argv);
 
        /* Where to log in inetd modes? "Classic" inetd
@@ -122,7 +122,7 @@ int fakeidentd_main(int argc, char **argv)
         * log to stderr. I like daemontools more. Go their way.
         * (Or maybe we need yet another option "log to syslog") */
        if (!(opt & OPT_fiw) /* || (opt & OPT_syslog) */) {
-               openlog(applet_name, 0, LOG_DAEMON);
+               openlog(applet_name, LOG_PID, LOG_DAEMON);
                logmode = LOGMODE_SYSLOG;
        }