CC: [email protected]
In-Reply-To: <[email protected]>
References: <[email protected]>
TO: Olga Kornievskaia <[email protected]>
TO: [email protected]
TO: [email protected]
CC: [email protected]
CC: [email protected]
CC: [email protected]

Hi Olga,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on nfs/linux-next]
[also build test WARNING on pcmoore-selinux/next linus/master 
security/next-testing v5.11 next-20210218]
[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]

url:    
https://github.com/0day-ci/linux/commits/Olga-Kornievskaia/Add-new-hook-to-compare-new-mount-to-an-existing-mount/20210219-035957
base:   git://git.linux-nfs.org/projects/trondmy/linux-nfs.git linux-next
:::::: branch date: 4 hours ago
:::::: commit date: 4 hours ago
config: i386-randconfig-m021-20210215 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
Reported-by: Dan Carpenter <[email protected]>

smatch warnings:
fs/nfs/super.c:1061 nfs_fill_super() error: we previously assumed 'ctx' could 
be null (see line 1029)

vim +/ctx +1061 fs/nfs/super.c

48b605f83c920d Jeff Layton       2008-06-10  1017  
f7b422b17ee5ee David Howells     2006-06-09  1018  /*
ab88dca311a372 Al Viro           2019-12-10  1019   * Finish setting up an NFS 
superblock
f7b422b17ee5ee David Howells     2006-06-09  1020   */
62a55d088cd87d Scott Mayhew      2019-12-10  1021  static void 
nfs_fill_super(struct super_block *sb, struct nfs_fs_context *ctx)
f7b422b17ee5ee David Howells     2006-06-09  1022  {
54ceac45159860 David Howells     2006-08-22  1023       struct nfs_server 
*server = NFS_SB(sb);
f7b422b17ee5ee David Howells     2006-06-09  1024  
f7b422b17ee5ee David Howells     2006-06-09  1025       sb->s_blocksize_bits = 
0;
f7b422b17ee5ee David Howells     2006-06-09  1026       sb->s_blocksize = 0;
6a74490dca8974 Bryan Schumaker   2012-07-30  1027       sb->s_xattr = 
server->nfs_client->cl_nfs_mod->xattr;
6a74490dca8974 Bryan Schumaker   2012-07-30  1028       sb->s_op = 
server->nfs_client->cl_nfs_mod->sops;
5eb005caf5383d David Howells     2019-12-10 @1029       if (ctx && ctx->bsize)
5eb005caf5383d David Howells     2019-12-10  1030               sb->s_blocksize 
= nfs_block_size(ctx->bsize, &sb->s_blocksize_bits);
f7b422b17ee5ee David Howells     2006-06-09  1031  
6a74490dca8974 Bryan Schumaker   2012-07-30  1032       if 
(server->nfs_client->rpc_ops->version != 2) {
54ceac45159860 David Howells     2006-08-22  1033               /* The VFS 
shouldn't apply the umask to mode bits. We will do
54ceac45159860 David Howells     2006-08-22  1034                * so ourselves 
when necessary.
54ceac45159860 David Howells     2006-08-22  1035                */
1751e8a6cb935e Linus Torvalds    2017-11-27  1036               sb->s_flags |= 
SB_POSIXACL;
54ceac45159860 David Howells     2006-08-22  1037               sb->s_time_gran 
= 1;
20fa1902728698 Peng Tao          2017-06-29  1038               sb->s_export_op 
= &nfs_export_ops;
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1039       } else
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1040               sb->s_time_gran 
= 1000;
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1041  
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1042       if 
(server->nfs_client->rpc_ops->version != 4) {
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1043               sb->s_time_min 
= 0;
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1044               sb->s_time_max 
= U32_MAX;
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1045       } else {
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1046               sb->s_time_min 
= S64_MIN;
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1047               sb->s_time_max 
= S64_MAX;
54ceac45159860 David Howells     2006-08-22  1048       }
f7b422b17ee5ee David Howells     2006-06-09  1049  
ab88dca311a372 Al Viro           2019-12-10  1050       sb->s_magic = 
NFS_SUPER_MAGIC;
54ceac45159860 David Howells     2006-08-22  1051  
ab88dca311a372 Al Viro           2019-12-10  1052       /* We probably want 
something more informative here */
ab88dca311a372 Al Viro           2019-12-10  1053       snprintf(sb->s_id, 
sizeof(sb->s_id),
ab88dca311a372 Al Viro           2019-12-10  1054                "%u:%u", 
MAJOR(sb->s_dev), MINOR(sb->s_dev));
1fcb79c1b21801 Deepa Dinamani    2019-03-26  1055  
ab88dca311a372 Al Viro           2019-12-10  1056       if (sb->s_blocksize == 
0)
ab88dca311a372 Al Viro           2019-12-10  1057               sb->s_blocksize 
= nfs_block_bits(server->wsize,
ab88dca311a372 Al Viro           2019-12-10  1058                               
                 &sb->s_blocksize_bits);
f7b422b17ee5ee David Howells     2006-06-09  1059  
ab88dca311a372 Al Viro           2019-12-10  1060       
nfs_super_set_maxbytes(sb, server->maxfilesize);
52a2a3a4af9af7 Olga Kornievskaia 2021-02-18 @1061       
server->has_sec_mnt_opts = ctx->has_sec_mnt_opts;
f7b422b17ee5ee David Howells     2006-06-09  1062  }
f7b422b17ee5ee David Howells     2006-06-09  1063  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to