]> git.ipfire.org Git - thirdparty/bird.git/commitdiff
Unix socket: Path length check directly before copying the path.
authorMaria Matejka <mq@ucw.cz>
Sat, 17 Aug 2019 12:57:41 +0000 (14:57 +0200)
committerMaria Matejka <mq@ucw.cz>
Sat, 17 Aug 2019 12:57:41 +0000 (14:57 +0200)
This is not needed as the string is always short enough, anyway
it may be needed in future and one strlen during BIRD start is
cheap enough.

sysdep/unix/io.c

index 8f76652ade592853f89da071014d14f7551bfc1a..596fc1eecac344209e9a8dca29305a9b7a61e971 100644 (file)
@@ -1493,7 +1493,9 @@ sk_open_unix(sock *s, char *name)
   if (fcntl(fd, F_SETFL, O_NONBLOCK) < 0)
     return -1;
 
-  /* Path length checked in test_old_bird() */
+  /* Path length checked in test_old_bird() but we may need unix sockets for other reasons in future */
+  ASSERT_DIE(strlen(name) < sizeof(sa.sun_path));
+
   sa.sun_family = AF_UNIX;
   strcpy(sa.sun_path, name);