On Sun, Mar 31, 2019 at 07:10:04PM +0300, Shamir Rabinovitch wrote: > @@ -788,6 +794,9 @@ void uverbs_close_fd(struct file *f) > { > struct ib_uobject *uobj = f->private_data; > struct ib_uverbs_file *ufile = uobj->ufile; > + struct uverbs_attr_bundle attrs = { > + .context = uobj->context, > + .ufile = uobj->context->ufile }; This will NULL derefence context in some cases, it should just be 'ufile' as we already have it on the stack. Jason