vfs: Remove the const from dir_context::actor
authorDavid Howells <dhowells@redhat.com>
Mon, 9 Apr 2018 20:12:30 +0000 (21:12 +0100)
committerDavid Howells <dhowells@redhat.com>
Mon, 9 Apr 2018 20:12:30 +0000 (21:12 +0100)
Remove the const marking from the actor function pointer in the dir_context
struct.  The const prevents the structure from being used as part of a
kmalloc'd object as it makes the compiler require that the actor member be
set at object initialisation time (or not at all), incuring something like
the following error if you try and set it later:

fs/afs/dir.c:556:20: error: assignment of read-only member 'actor'

Marking the member const like this adds very little in the way of sanity
checking as the type checking system is likely to provide sufficient - and
if not, the kernel is very likely to oops repeatably in this case.

Fixes: ac6614b76478 ("[readdir] constify ->actor")
Signed-off-by: David Howells <dhowells@redhat.com>
Reviewed-by: Al Viro <viro@zeniv.linux.org.uk>
include/linux/fs.h

index 1ee7f592e239c3b11587f738b42b06b05291e348..3a5c19d9f651aab263f44703001cba8b9a77ca58 100644 (file)
@@ -1667,7 +1667,7 @@ typedef int (*filldir_t)(struct dir_context *, const char *, int, loff_t, u64,
                         unsigned);
 
 struct dir_context {
-       const filldir_t actor;
+       filldir_t actor;
        loff_t pos;
 };