On Monday, 19 February 2024 16:38:06 CDT Elizabeth Figura wrote: > +static struct ntsync_obj *get_obj(struct ntsync_device *dev, int fd) > +{ > + struct file *file = fget(fd); > + struct ntsync_obj *obj; > + > + if (file->f_op != &ntsync_obj_fops) { > + fput(file); > + return NULL; > + } I just noticed during self-review that this fails to check the result of fget() for NULL :-/ I'll fix this next revision. > + > + obj = file->private_data; > + if (obj->dev != dev) { > + fput(file); > + return NULL; > + } > + > + return obj; > +}