chiark / gitweb /
fix off-by-one in pass_env_to_socket()
authorAlan Jenkins <alan-jenkins@tuffmail.co.uk>
Mon, 8 Sep 2008 22:48:17 +0000 (00:48 +0200)
committerKay Sievers <kay.sievers@vrfy.org>
Mon, 8 Sep 2008 22:48:17 +0000 (00:48 +0200)
udev/udev_rules.c

index 004a89f..3b9631b 100644 (file)
@@ -486,10 +486,10 @@ static int pass_env_to_socket(struct udev *udev, const char *sockpath, const cha
                saddrlen = offsetof(struct sockaddr_un, sun_path) + 1 + strlen(&saddr.sun_path[1]);
        }
 
-       bufpos = snprintf(buf, sizeof(buf)-1, "%s@%s", action, devpath);
+       bufpos = snprintf(buf, sizeof(buf), "%s@%s", action, devpath);
        bufpos++;
-       for (i = 0; environ[i] != NULL && bufpos < (sizeof(buf)-1); i++) {
-               bufpos += strlcpy(&buf[bufpos], environ[i], sizeof(buf) - bufpos-1);
+       for (i = 0; environ[i] != NULL && bufpos < (sizeof(buf)); i++) {
+               bufpos += strlcpy(&buf[bufpos], environ[i], sizeof(buf) - bufpos);
                bufpos++;
        }
        if (bufpos > sizeof(buf))