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;
> +}




Reply via email to