]> www.infradead.org Git - users/hch/xfstests-dev.git/commitdiff
src/attr_replace_test: limit size of extended attribute value
authorLuis Henriques <lhenriques@suse.com>
Tue, 21 May 2019 08:28:36 +0000 (09:28 +0100)
committerEryu Guan <guaneryu@gmail.com>
Thu, 23 May 2019 08:11:55 +0000 (16:11 +0800)
The maximum size for extended attribute values is 65536 (XATTR_SIZE_MAX).
Since there are filesystems that can set blksize to really big values
(CephFS for example has a default of 4M), it's easy to have this test
failing with fsetxattr returning -E2BIG.

Cc: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Luis Henriques <lhenriques@suse.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
src/attr_replace_test.c

index 0720bfdc18ab61749ccf663914f894eda0f1fd0c..cca8dcf8ff6014fd2c2fc300d7496807117f2fdd 100644 (file)
@@ -9,6 +9,8 @@
 #include <sys/types.h>
 #include <sys/xattr.h>
 #include <sys/stat.h>
+#include <sys/param.h>
+#include <linux/limits.h>
 
 #define die() do { perror(""); \
 fprintf(stderr, "error at line %d\n", __LINE__); \
@@ -44,6 +46,7 @@ int main(int argc, char *argv[])
        size = sbuf.st_blksize * 3 / 4;
        if (!size)
                fail("Invalid st_blksize(%ld)\n", sbuf.st_blksize);
+       size = MIN(size, XATTR_SIZE_MAX);
        value = malloc(size);
        if (!value)
                fail("Failed to allocate memory\n");