This caused a kernel oops since %pdH interpreted the pointer as a struct file.
Signed-off-by: Hans Verkuil hans.verkuil@cisco.com --- drivers/gpu/drm/drm_dp_cec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_dp_cec.c b/drivers/gpu/drm/drm_dp_cec.c index 87b67cc1ea58..a6cac47f6248 100644 --- a/drivers/gpu/drm/drm_dp_cec.c +++ b/drivers/gpu/drm/drm_dp_cec.c @@ -157,7 +157,7 @@ static void drm_dp_cec_adap_status(struct cec_adapter *adap,
if (drm_dp_read_desc(aux, &desc, true)) return; - seq_printf(file, "OUI: %*pdH\n", + seq_printf(file, "OUI: %*phD\n", (int)sizeof(id->oui), id->oui); seq_printf(file, "ID: %*pE\n", (int)strnlen(id->device_id, sizeof(id->device_id)),
On Tue, Jul 24, 2018 at 09:20:28PM +0200, Hans Verkuil wrote:
This caused a kernel oops since %pdH interpreted the pointer as a struct file.
Signed-off-by: Hans Verkuil hans.verkuil@cisco.com
drivers/gpu/drm/drm_dp_cec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_dp_cec.c b/drivers/gpu/drm/drm_dp_cec.c index 87b67cc1ea58..a6cac47f6248 100644 --- a/drivers/gpu/drm/drm_dp_cec.c +++ b/drivers/gpu/drm/drm_dp_cec.c @@ -157,7 +157,7 @@ static void drm_dp_cec_adap_status(struct cec_adapter *adap,
if (drm_dp_read_desc(aux, &desc, true)) return;
- seq_printf(file, "OUI: %*pdH\n",
- seq_printf(file, "OUI: %*phD\n", (int)sizeof(id->oui), id->oui); seq_printf(file, "ID: %*pE\n", (int)strnlen(id->device_id, sizeof(id->device_id)),
pushed to drm-misc-next-fixes for 4.19. Thanks.
Gustavo
dri-devel@lists.freedesktop.org