chiark / gitweb /
util: add allocation loop to gettyname_malloc()
authorLennart Poettering <lennart@poettering.net>
Tue, 23 Dec 2014 01:10:08 +0000 (02:10 +0100)
committerLennart Poettering <lennart@poettering.net>
Tue, 23 Dec 2014 02:26:24 +0000 (03:26 +0100)
src/shared/util.c

index 6695a85a1c904af9be5e03ef6af90de1d5025f68..6bd278e7901a888fedd6c946c3f97130711a76c7 100644 (file)
@@ -2785,23 +2785,36 @@ char *getusername_malloc(void) {
         return lookup_uid(getuid());
 }
 
-int getttyname_malloc(int fd, char **r) {
-        char path[PATH_MAX], *c;
-        int k;
+int getttyname_malloc(int fd, char **ret) {
+        size_t l = 100;
+        int r;
 
-        assert(r);
+        assert(fd >= 0);
+        assert(ret);
 
-        k = ttyname_r(fd, path, sizeof(path));
-        if (k > 0)
-                return -k;
+        for (;;) {
+                char path[l];
 
-        char_array_0(path);
+                r = ttyname_r(fd, path, sizeof(path));
+                if (r == 0) {
+                        const char *p;
+                        char *c;
 
-        c = strdup(startswith(path, "/dev/") ? path + 5 : path);
-        if (!c)
-                return -ENOMEM;
+                        p = startswith(path, "/dev/");
+                        c = strdup(p ?: path);
+                        if (!c)
+                                return -ENOMEM;
+
+                        *ret = c;
+                        return 0;
+                }
+
+                if (r != ERANGE)
+                        return -r;
+
+                l *= 2;
+        }
 
-        *r = c;
         return 0;
 }
 
@@ -7437,6 +7450,9 @@ int sethostname_idempotent(const char *s) {
 int ptsname_malloc(int fd, char **ret) {
         size_t l = 100;
 
+        assert(fd >= 0);
+        assert(ret);
+
         for (;;) {
                 char *c;