diff options
author | Kees Cook <keescook@chromium.org> | 2014-09-11 13:53:54 -0700 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2014-09-17 11:15:01 +1000 |
commit | 109ab90974995a06aeabab1535b0ce00f26dd24a (patch) | |
tree | d8c0a8c7ae5a783b0a484a9700c30604ec65c041 /drivers/gpu | |
parent | d898ce03675fc061f89a347a22d41271ed75c436 (diff) | |
download | linux-109ab90974995a06aeabab1535b0ce00f26dd24a.tar.bz2 |
drm/ttm: make sure format string cannot leak in
While zone->name is currently hard coded, the call to kobject_init_and_add()
should follow the more defensive argument list usage (as already done in
other places in ttm_memory.c) where "%s" is used instead of directly passing
in a variable as a format string.
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r-- | drivers/gpu/drm/ttm/ttm_memory.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_memory.c b/drivers/gpu/drm/ttm/ttm_memory.c index dbc2def887cd..a1803fbcc898 100644 --- a/drivers/gpu/drm/ttm/ttm_memory.c +++ b/drivers/gpu/drm/ttm/ttm_memory.c @@ -300,7 +300,8 @@ static int ttm_mem_init_highmem_zone(struct ttm_mem_global *glob, zone->glob = glob; glob->zone_highmem = zone; ret = kobject_init_and_add( - &zone->kobj, &ttm_mem_zone_kobj_type, &glob->kobj, zone->name); + &zone->kobj, &ttm_mem_zone_kobj_type, &glob->kobj, "%s", + zone->name); if (unlikely(ret != 0)) { kobject_put(&zone->kobj); return ret; |