diff options
Diffstat (limited to 'src/shared')
| -rw-r--r-- | src/shared/macro.h | 17 | ||||
| -rw-r--r-- | src/shared/util.c | 5 |
2 files changed, 20 insertions, 2 deletions
diff --git a/src/shared/macro.h b/src/shared/macro.h index 9bf81dc3cc..f91f7de769 100644 --- a/src/shared/macro.h +++ b/src/shared/macro.h @@ -264,6 +264,23 @@ do { \ } \ } while(false) +/* Remove this macro, when the kernel has f_type as unsigned int or long + * for every architecure. Currently some 64bit architecures (like s390x) + * have int in the kernel, but long in userspace for f_type, so glibc + * extends the int to long and carries over the sign. Negative numbers are + * caused by the 32bit magic constants in linux/magic.h stuffed into the + * signed int in the kernel and these negative numbers are extended to + * long, which cannot be simply compared to the magic constants anymore. + */ +#define CMP_F_TYPE(f, c) \ + __extension__ ({ \ + __SWORD_TYPE _f = (f); \ + const __SWORD_TYPE _c = (c); \ + const int _c32 = (c); \ + (_f == _c || _f == _c32 ); \ + }) + + /* Returns the number of chars needed to format variables of the * specified type as a decimal string. Adds in extra space for a * negative '-' prefix. */ diff --git a/src/shared/util.c b/src/shared/util.c index 386973b3fc..2a8046922b 100644 --- a/src/shared/util.c +++ b/src/shared/util.c @@ -2779,8 +2779,9 @@ int rm_rf_children_dangerous(int fd, bool only_dirs, bool honour_sticky, struct static int is_temporary_fs(struct statfs *s) { assert(s); - return (unsigned __SWORD_TYPE) s->f_type == TMPFS_MAGIC || - (unsigned __SWORD_TYPE) s->f_type == RAMFS_MAGIC; + return + CMP_F_TYPE(s->f_type, TMPFS_MAGIC) || + CMP_F_TYPE(s->f_type, RAMFS_MAGIC); } int rm_rf_children(int fd, bool only_dirs, bool honour_sticky, struct stat *root_dev) { |