]> www.infradead.org Git - linux.git/commitdiff
uio: Constify struct kobj_type
authorHongbo Li <lihongbo22@huawei.com>
Wed, 4 Sep 2024 01:22:00 +0000 (09:22 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 11 Sep 2024 14:02:54 +0000 (16:02 +0200)
These 'struct kobj_type' are not modified. They are only used in
kobject_init() which takes a 'const struct kobj_type *ktype'
parameter.

Constifying these structure and moving them to a read-only
section (from data to text), and can increase over all security.

```
[Before]
   text   data    bss    dec    hex    filename
  10330   1908     20  12258   2fe2    drivers/uio/uio.o

[After]
  text    data    bss    dec    hex    filename
  10458   1844     20  12322   3022    drivers/uio/uio.o
```

Signed-off-by: Hongbo Li <lihongbo22@huawei.com>
Link: https://lore.kernel.org/r/20240904012200.2010916-1-lihongbo22@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/uio/uio.c

index 20d2a55cb40b874cdba104fb5d5f09b6639d5d87..004a549c6c7d42c1ac12bfea9fd4c99f3868369e 100644 (file)
@@ -118,7 +118,7 @@ static const struct sysfs_ops map_sysfs_ops = {
        .show = map_type_show,
 };
 
-static struct kobj_type map_attr_type = {
+static const struct kobj_type map_attr_type = {
        .release        = map_release,
        .sysfs_ops      = &map_sysfs_ops,
        .default_groups = map_groups,
@@ -207,7 +207,7 @@ static const struct sysfs_ops portio_sysfs_ops = {
        .show = portio_type_show,
 };
 
-static struct kobj_type portio_attr_type = {
+static const struct kobj_type portio_attr_type = {
        .release        = portio_release,
        .sysfs_ops      = &portio_sysfs_ops,
        .default_groups = portio_groups,