Hi Khadija, kernel test robot noticed the following build errors: [auto build test ERROR on pcmoore-selinux/next] [also build test ERROR on pcmoore-audit/next linus/master v6.5-rc7 next-20230823] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Khadija-Kamran/lsm-constify-mnt_opts-parameter-in-security_free_mnt_opts/20230823-170310 base: https://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/selinux.git next patch link: https://lore.kernel.org/r/ZOXG%2FVVC1XRc7TtI%40gmail.com patch subject: [PATCH] lsm: constify 'mnt_opts' parameter in security_free_mnt_opts() config: riscv-randconfig-001-20230823 (https://download.01.org/0day-ci/archive/20230824/202308240128.Gsn0b0Fv-lkp@xxxxxxxxx/config) compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project.git f28c006a5895fc0e329fe15fead81e37457cb1d1) reproduce: (https://download.01.org/0day-ci/archive/20230824/202308240128.Gsn0b0Fv-lkp@xxxxxxxxx/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202308240128.Gsn0b0Fv-lkp@xxxxxxxxx/ All errors (new ones prefixed by >>): >> fs/btrfs/super.c:1523:25: error: passing 'void **' to parameter of type 'const void **' discards qualifiers in nested pointer types [-Werror,-Wincompatible-pointer-types-discards-qualifiers] security_free_mnt_opts(&new_sec_opts); ^~~~~~~~~~~~~ include/linux/security.h:301:42: note: passing argument to parameter 'mnt_opts' here void security_free_mnt_opts(const void **mnt_opts); ^ fs/btrfs/super.c:1536:25: error: passing 'void **' to parameter of type 'const void **' discards qualifiers in nested pointer types [-Werror,-Wincompatible-pointer-types-discards-qualifiers] security_free_mnt_opts(&new_sec_opts); ^~~~~~~~~~~~~ include/linux/security.h:301:42: note: passing argument to parameter 'mnt_opts' here void security_free_mnt_opts(const void **mnt_opts); ^ fs/btrfs/super.c:1699:26: error: passing 'void **' to parameter of type 'const void **' discards qualifiers in nested pointer types [-Werror,-Wincompatible-pointer-types-discards-qualifiers] security_free_mnt_opts(&new_sec_opts); ^~~~~~~~~~~~~ include/linux/security.h:301:42: note: passing argument to parameter 'mnt_opts' here void security_free_mnt_opts(const void **mnt_opts); ^ 3 errors generated. vim +1523 fs/btrfs/super.c 450ba0ea06b6ed Josef Bacik 2010-11-19 1427 312c89fbca0689 Misono, Tomohiro 2017-12-14 1428 /* 312c89fbca0689 Misono, Tomohiro 2017-12-14 1429 * Find a superblock for the given device / mount point. 312c89fbca0689 Misono, Tomohiro 2017-12-14 1430 * 312c89fbca0689 Misono, Tomohiro 2017-12-14 1431 * Note: This is based on mount_bdev from fs/super.c with a few additions 312c89fbca0689 Misono, Tomohiro 2017-12-14 1432 * for multiple device setup. Make sure to keep it in sync. 312c89fbca0689 Misono, Tomohiro 2017-12-14 1433 */ 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1434 static struct dentry *btrfs_mount_root(struct file_system_type *fs_type, 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1435 int flags, const char *device_name, void *data) 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1436 { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1437 struct block_device *bdev = NULL; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1438 struct super_block *s; 36350e95a2b1fe Gu Jinxiang 2018-07-12 1439 struct btrfs_device *device = NULL; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1440 struct btrfs_fs_devices *fs_devices = NULL; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1441 struct btrfs_fs_info *fs_info = NULL; 204cc0ccf1d49c Al Viro 2018-12-13 1442 void *new_sec_opts = NULL; 05bdb9965305bb Christoph Hellwig 2023-06-08 1443 blk_mode_t mode = sb_open_mode(flags); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1444 int error = 0; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1445 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1446 if (data) { a65001e8a4d465 Al Viro 2018-12-10 1447 error = security_sb_eat_lsm_opts(data, &new_sec_opts); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1448 if (error) 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1449 return ERR_PTR(error); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1450 } 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1451 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1452 /* 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1453 * Setup a dummy root and fs_info for test/set super. This is because 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1454 * we don't actually fill this stuff out until open_ctree, but we need 8260edba67a2e6 Josef Bacik 2020-01-24 1455 * then open_ctree will properly initialize the file system specific 8260edba67a2e6 Josef Bacik 2020-01-24 1456 * settings later. btrfs_init_fs_info initializes the static elements 8260edba67a2e6 Josef Bacik 2020-01-24 1457 * of the fs_info (locks and such) to make cleanup easier if we find a 8260edba67a2e6 Josef Bacik 2020-01-24 1458 * superblock with our given fs_devices later on at sget() time. 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1459 */ a8fd1f71749387 Jeff Mahoney 2018-02-15 1460 fs_info = kvzalloc(sizeof(struct btrfs_fs_info), GFP_KERNEL); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1461 if (!fs_info) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1462 error = -ENOMEM; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1463 goto error_sec_opts; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1464 } 8260edba67a2e6 Josef Bacik 2020-01-24 1465 btrfs_init_fs_info(fs_info); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1466 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1467 fs_info->super_copy = kzalloc(BTRFS_SUPER_INFO_SIZE, GFP_KERNEL); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1468 fs_info->super_for_commit = kzalloc(BTRFS_SUPER_INFO_SIZE, GFP_KERNEL); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1469 if (!fs_info->super_copy || !fs_info->super_for_commit) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1470 error = -ENOMEM; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1471 goto error_fs_info; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1472 } 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1473 399f7f4c42e8a5 David Sterba 2018-06-19 1474 mutex_lock(&uuid_mutex); 2ef789288afd36 Christoph Hellwig 2023-06-08 1475 error = btrfs_parse_device_options(data, mode); 81ffd56b574535 David Sterba 2018-06-19 1476 if (error) { 399f7f4c42e8a5 David Sterba 2018-06-19 1477 mutex_unlock(&uuid_mutex); 399f7f4c42e8a5 David Sterba 2018-06-19 1478 goto error_fs_info; 81ffd56b574535 David Sterba 2018-06-19 1479 } 399f7f4c42e8a5 David Sterba 2018-06-19 1480 2ef789288afd36 Christoph Hellwig 2023-06-08 1481 device = btrfs_scan_one_device(device_name, mode); 36350e95a2b1fe Gu Jinxiang 2018-07-12 1482 if (IS_ERR(device)) { 399f7f4c42e8a5 David Sterba 2018-06-19 1483 mutex_unlock(&uuid_mutex); 36350e95a2b1fe Gu Jinxiang 2018-07-12 1484 error = PTR_ERR(device); 399f7f4c42e8a5 David Sterba 2018-06-19 1485 goto error_fs_info; 81ffd56b574535 David Sterba 2018-06-19 1486 } 399f7f4c42e8a5 David Sterba 2018-06-19 1487 36350e95a2b1fe Gu Jinxiang 2018-07-12 1488 fs_devices = device->fs_devices; 399f7f4c42e8a5 David Sterba 2018-06-19 1489 fs_info->fs_devices = fs_devices; 399f7f4c42e8a5 David Sterba 2018-06-19 1490 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1491 error = btrfs_open_devices(fs_devices, mode, fs_type); f5194e34cabadd David Sterba 2018-06-19 1492 mutex_unlock(&uuid_mutex); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1493 if (error) 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1494 goto error_fs_info; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1495 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1496 if (!(flags & SB_RDONLY) && fs_devices->rw_devices == 0) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1497 error = -EACCES; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1498 goto error_close_devices; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1499 } 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1500 d24fa5c1da0802 Anand Jain 2021-08-24 1501 bdev = fs_devices->latest_dev->bdev; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1502 s = sget(fs_type, btrfs_test_super, btrfs_set_super, flags | SB_NOSEC, 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1503 fs_info); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1504 if (IS_ERR(s)) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1505 error = PTR_ERR(s); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1506 goto error_close_devices; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1507 } 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1508 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1509 if (s->s_root) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1510 btrfs_close_devices(fs_devices); 0d4b0463011de0 Josef Bacik 2020-01-24 1511 btrfs_free_fs_info(fs_info); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1512 if ((flags ^ s->s_flags) & SB_RDONLY) 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1513 error = -EBUSY; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1514 } else { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1515 snprintf(s->s_id, sizeof(s->s_id), "%pg", bdev); e33c267ab70de4 Roman Gushchin 2022-05-31 1516 shrinker_debugfs_rename(&s->s_shrink, "sb-%s:%s", fs_type->name, e33c267ab70de4 Roman Gushchin 2022-05-31 1517 s->s_id); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1518 btrfs_sb(s)->bdev_holder = fs_type; 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1519 error = btrfs_fill_super(s, fs_devices, data); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1520 } a65001e8a4d465 Al Viro 2018-12-10 1521 if (!error) 204cc0ccf1d49c Al Viro 2018-12-13 1522 error = security_sb_set_mnt_opts(s, new_sec_opts, 0, NULL); a65001e8a4d465 Al Viro 2018-12-10 @1523 security_free_mnt_opts(&new_sec_opts); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1524 if (error) { 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1525 deactivate_locked_super(s); a65001e8a4d465 Al Viro 2018-12-10 1526 return ERR_PTR(error); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1527 } 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1528 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1529 return dget(s->s_root); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1530 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1531 error_close_devices: 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1532 btrfs_close_devices(fs_devices); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1533 error_fs_info: 0d4b0463011de0 Josef Bacik 2020-01-24 1534 btrfs_free_fs_info(fs_info); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1535 error_sec_opts: 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1536 security_free_mnt_opts(&new_sec_opts); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1537 return ERR_PTR(error); 72fa39f5c7a1c9 Misono, Tomohiro 2017-12-14 1538 } 312c89fbca0689 Misono, Tomohiro 2017-12-14 1539 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
