diff options
author | Luca Boccassi <bluca@debian.org> | 2023-03-20 01:40:20 +0000 |
---|---|---|
committer | Simon McVittie <smcv@collabora.com> | 2023-05-15 19:06:54 +0000 |
commit | 4364b9ebb3acef566a2c1955423426584147226e (patch) | |
tree | afbef4a885ef77ca7c0adcaf0e8b87912fc8f10e /dbus | |
parent | 1a74c43c0e545f41b6266713362c42748ee579e8 (diff) | |
download | dbus-4364b9ebb3acef566a2c1955423426584147226e.tar.gz |
Support /proc in _dbus_file_get_content
procfs has special semantics: most files are 0 size,
only one read can be done on a file, and they are
not larger than 4MB. Enhance _dbus_file_get_content()
so that we can read files from /proc with it.
Signed-off-by: Luca Boccassi <bluca@debian.org>
Diffstat (limited to 'dbus')
-rw-r--r-- | dbus/dbus-file-unix.c | 50 |
1 files changed, 46 insertions, 4 deletions
diff --git a/dbus/dbus-file-unix.c b/dbus/dbus-file-unix.c index 6da44504..d5fa029e 100644 --- a/dbus/dbus-file-unix.c +++ b/dbus/dbus-file-unix.c @@ -33,7 +33,13 @@ #include "dbus-sysdeps.h" #include "dbus-sysdeps-unix.h" +#ifdef HAVE_LINUX_MAGIC_H +#include <linux/magic.h> +#endif #include <sys/stat.h> +#ifdef HAVE_SYS_VFS_H +#include <sys/vfs.h> +#endif #include <stdio.h> #include <fcntl.h> #include <unistd.h> @@ -60,9 +66,14 @@ _dbus_file_get_contents (DBusString *str, { int fd; struct stat sb; +#ifdef HAVE_FSTATFS + struct statfs sfs; +#endif int orig_len; int total; + int file_size; const char *filename_c; + dbus_bool_t is_procfs = FALSE; _DBUS_ASSERT_ERROR_IS_CLEAR (error); @@ -104,17 +115,47 @@ _dbus_file_get_contents (DBusString *str, _dbus_close (fd, NULL); return FALSE; } + + /* procfs has different semantics - most files are 0 size, + * we can do only one read, and at most we can read 4M. + */ +#ifdef HAVE_FSTATFS + if (sb.st_size == 0) + { + if (fstatfs(fd, &sfs) < 0) + { + dbus_set_error (error, _dbus_error_from_errno (errno), + "Failed to stat \"%s\": %s", + filename_c, + _dbus_strerror (errno)); + + _dbus_verbose ("fstatvfs() failed: %s", + _dbus_strerror (errno)); + + _dbus_close (fd, NULL); + + return FALSE; + } + if (sfs.f_type == PROC_SUPER_MAGIC) + is_procfs = TRUE; + } +#endif + + if (is_procfs) + file_size = _DBUS_ONE_MEGABYTE; + else + file_size = sb.st_size; total = 0; orig_len = _dbus_string_get_length (str); - if (sb.st_size > 0 && S_ISREG (sb.st_mode)) + if (file_size > 0 && S_ISREG (sb.st_mode)) { int bytes_read; - while (total < (int) sb.st_size) + do { bytes_read = _dbus_read (fd, str, - sb.st_size - total); + file_size - total); if (bytes_read <= 0) { dbus_set_error (error, _dbus_error_from_errno (errno), @@ -132,11 +173,12 @@ _dbus_file_get_contents (DBusString *str, else total += bytes_read; } + while (total < file_size && !is_procfs); _dbus_close (fd, NULL); return TRUE; } - else if (sb.st_size != 0) + else if (file_size != 0) { _dbus_verbose ("Can only open regular files at the moment.\n"); dbus_set_error (error, DBUS_ERROR_FAILED, |