Geforce File Manager

NAME	SIZE	ACTIONS
📁 alloc		[ DEL ]
📁 debugfs		[ DEL ]
📄 check_bq27xxx_data.cocci	3,504 B	[ EDIT ] \| [ DEL ]
📄 d_find_alias.cocci	1,309 B	[ EDIT ] \| [ DEL ]
📄 drm-get-put.cocci	1,532 B	[ EDIT ] \| [ DEL ]
📄 err_cast.cocci	1,097 B	[ EDIT ] \| [ DEL ]
📄 kstrdup.cocci	2,461 B	[ EDIT ] \| [ DEL ]
📄 memdup.cocci	1,350 B	[ EDIT ] \| [ DEL ]
📄 memdup_user.cocci	1,306 B	[ EDIT ] \| [ DEL ]
📄 platform_no_drv_owner.cocci	3,869 B	[ EDIT ] \| [ DEL ]
📄 pm_runtime.cocci	2,394 B	[ EDIT ] \| [ DEL ]
📄 ptr_ret.cocci	1,814 B	[ EDIT ] \| [ DEL ]
📄 resource_size.cocci	2,115 B	[ EDIT ] \| [ DEL ]
📄 simple_open.cocci	1,244 B	[ EDIT ] \| [ DEL ]
📄 vma_pages.cocci	1,519 B	[ EDIT ] \| [ DEL ]

[ CLOSE ]

EDIT: simple_open.cocci

// SPDX-License-Identifier: GPL-2.0
/// Remove an open coded simple_open() function
/// and replace file operations references to the function
/// with simple_open() instead.
///
// Confidence: High
// Comments:
// Options: --no-includes --include-headers

virtual patch
virtual report

@ open depends on patch @
identifier open_f != simple_open;
identifier i, f;
@@
-int open_f(struct inode *i, struct file *f)
-{
(
-if (i->i_private)
-f->private_data = i->i_private;
|
-f->private_data = i->i_private;
)
-return 0;
-}

@ has_open depends on open @
identifier fops;
identifier open.open_f;
@@
struct file_operations fops = {
...,
-.open = open_f,
+.open = simple_open,
...
};

@ openr depends on report @
identifier open_f != simple_open;
identifier i, f;
position p;
@@
int open_f@p(struct inode *i, struct file *f)
{
(
if (i->i_private)
f->private_data = i->i_private;
|
f->private_data = i->i_private;
)
return 0;
}

@ has_openr depends on openr @
identifier fops;
identifier openr.open_f;
position p;
@@
struct file_operations fops = {
...,
.open = open_f@p,
...
};

@script:python@
pf << openr.p;
ps << has_openr.p;
@@

coccilib.report.print_report(pf[0],"WARNING opportunity for simple_open, see also structure on line %s"%(ps[0].line))