fix build regression in oldmalloc

[musl] / src / passwd / nscd_query.c

diff --git a/src/passwd/nscd_query.c b/src/passwd/nscd_query.c
index 55ccc0a..d38e371 100644 (file)
--- a/src/passwd/nscd_query.c
+++ b/src/passwd/nscd_query.c
@@ -4,6 +4,7 @@
 #include <stdio.h>
 #include <string.h>
 #include <errno.h>
+#include <limits.h>
 #include "nscd.h"
 
 static const struct {
@@ -22,7 +23,7 @@ FILE *__nscd_query(int32_t req, const char *key, int32_t *buf, size_t len, int *
        int32_t req_buf[REQ_LEN] = {
                NSCDVERSION,
                req,
-               strlen(key)+1
+               strnlen(key,LOGIN_NAME_MAX)+1
        };
        struct msghdr msg = {
                .msg_iov = (struct iovec[]){
@@ -31,6 +32,7 @@ FILE *__nscd_query(int32_t req, const char *key, int32_t *buf, size_t len, int *
                },
                .msg_iovlen = 2
        };
+       int errno_save = errno;
 
        *swap = 0;
 retry:
@@ -45,15 +47,18 @@ retry:
                return 0;
        }
 
-       if (strlen(key) > INT32_MAX - 1)
+       if (req_buf[2] > LOGIN_NAME_MAX)
                return f;
 
        if (connect(fd, (struct sockaddr*)&addr, sizeof(addr)) < 0) {
-               /* If there isn't a running nscd we return -1 to indicate that
-                * that is precisely what happened
-                */
-               if (errno == EACCES || errno == ECONNREFUSED || errno == ENOENT)
+               /* If there isn't a running nscd we simulate a "not found"
+                * result and the caller is responsible for calling
+                * fclose on the (unconnected) socket. The value of
+                * errno must be left unchanged in this case.  */
+               if (errno == EACCES || errno == ECONNREFUSED || errno == ENOENT) {
+                       errno = errno_save;
                        return f;
+               }
                goto error;
        }