11 Mar
2024
11 Mar
'24
6:58 p.m.
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; +}