diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-08-04 16:32:39 +0200 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2022-09-07 10:42:28 +0200 |
commit | 184233a5202786b20220acd2d04ddf909ef18f29 (patch) | |
tree | 87e2ab8062ecb66030ab910f7bac198a8623ae56 /drivers/iommu/mtk_iommu.c | |
parent | Linux 6.0-rc4 (diff) | |
download | linux-184233a5202786b20220acd2d04ddf909ef18f29.tar.xz linux-184233a5202786b20220acd2d04ddf909ef18f29.zip |
iommu/omap: Fix buffer overflow in debugfs
There are two issues here:
1) The "len" variable needs to be checked before the very first write.
Otherwise if omap2_iommu_dump_ctx() with "bytes" less than 32 it is a
buffer overflow.
2) The snprintf() function returns the number of bytes that *would* have
been copied if there were enough space. But we want to know the
number of bytes which were *actually* copied so use scnprintf()
instead.
Fixes: bd4396f09a4a ("iommu/omap: Consolidate OMAP IOMMU modules")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Robin Murphy <robin.murphy@arm.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Link: https://lore.kernel.org/r/YuvYh1JbE3v+abd5@kili
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu/mtk_iommu.c')
0 files changed, 0 insertions, 0 deletions