Re: [PATCH] dma-buf/sync_file: Use fdget()

[Christian König]

Am 05.05.23 um 05:03 schrieb ye.xingc...@zte.com.cn:

From: Ye Xingchen <ye.xingc...@zte.com.cn>

convert the fget() use to fdget().

Well the rational is missing. Why should we do that?

Christian.

Signed-off-by: Ye Xingchen <ye.xingc...@zte.com.cn>
---
  drivers/dma-buf/sync_file.c | 10 +++++-----
  1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/dma-buf/sync_file.c b/drivers/dma-buf/sync_file.c
index af57799c86ce..222b13b1bdb8 100644
--- a/drivers/dma-buf/sync_file.c
+++ b/drivers/dma-buf/sync_file.c
@@ -78,18 +78,18 @@ EXPORT_SYMBOL(sync_file_create);

  static struct sync_file *sync_file_fdget(int fd)
  {
-       struct file *file = fget(fd);
+       struct struct fd f = fdget(fd);

-       if (!file)
+       if (!f.file)
                return NULL;

-       if (file->f_op != &sync_file_fops)
+       if (f.file->f_op != &sync_file_fops)
                goto err;

-       return file->private_data;
+       return f.file->private_data;

  err:
-       fput(file);
+       fdput(f);
        return NULL;
  }