ACPI: pfr_telemetry: Fix info leak in pfrt_log_ioctl()

The "data_info" struct is copied to the user.  It has a 4 byte struct
hole after the last struct member so we need to memset that to avoid
copying uninitialized stack data to the user.

Fixes: bb1c75a5f7c0 ("ACPI: Introduce Platform Firmware Runtime Telemetry driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

diff --git a/drivers/acpi/pfr_telemetry.c b/drivers/acpi/pfr_telemetry.c
index da50dd80192c9676c8716f49a8947d11b83009b6..9abf350bd7a5a28cc63d8960c395b02a0482b240 100644 (file)
--- a/drivers/acpi/pfr_telemetry.c
+++ b/drivers/acpi/pfr_telemetry.c
@@ -83,6 +83,7 @@ static int get_pfrt_log_data_info(struct pfrt_log_data_info *data_info,
        union acpi_object *out_obj, in_obj, in_buf;
        int ret = -EBUSY;
 
+       memset(data_info, 0, sizeof(*data_info));
        memset(&in_obj, 0, sizeof(in_obj));
        memset(&in_buf, 0, sizeof(in_buf));
        in_obj.type = ACPI_TYPE_PACKAGE;