[PATCH] nul-terminate readlink result

[Tobias Stoeckmann]

On Sat, Jul 11, 2015 at 11:02:57AM +0200, Tobias Stoeckmann wrote:
>       if (readlink(path, link, PATH_SIZE) < 0)
>               return -EINVAL;
> +     link[PATH_SIZE] = '\0';

Sorry, this patch is wrong. It has to be terminated at the correct
position. Please see this updated version:


>From a1840916dd370d5c84ebf23df91f1c13d564d212 Mon Sep 17 00:00:00 2001
From: Tobias Stoeckmann <tob...@stoeckmann.org>
Date: Sun, 12 Jul 2015 12:10:59 +0200
Subject: [PATCH] nul-terminate readlink result

readlink by itself does not terminate its result. The caller has
to terminate the string at the proper position.
---
 libkms/linux.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/libkms/linux.c b/libkms/linux.c
index 4d47148..5e2431f 100644
--- a/libkms/linux.c
+++ b/libkms/linux.c
@@ -55,6 +55,7 @@ linux_name_from_sysfs(int fd, char **out)
        unsigned maj, min;
        char* slash_name;
        int ret;
+       ssize_t len;

        /* 
         * Inside the sysfs directory for the device there is a symlink
@@ -80,8 +81,9 @@ linux_name_from_sysfs(int fd, char **out)

        snprintf(path, PATH_SIZE, "/sys/dev/char/%d:%d/device/driver", maj, 
min);

-       if (readlink(path, link, PATH_SIZE) < 0)
+       if ((len = readlink(path, link, PATH_SIZE)) < 0)
                return -EINVAL;
+       link[len] = '\0';

        /* link looks something like this: ../../../bus/pci/drivers/intel */
        slash_name = strrchr(link, '/');
-- 
2.4.5