In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
HID: nintendo: fix out-of-bounds read in joycon_ctlr_read_handler()<\/p>\n
joycon_ctlr_read_handler() casts an incoming HID input report to
\nstruct joycon_input_report and parses it, guarding the cast only with a
\n12-byte length check:<\/p>\n
\tif (size >= 12) \/* make sure it contains the input report *\/
\n\t\tjoycon_parse_report(ctlr, (struct joycon_input_report *)data);<\/p>\n
struct joycon_input_report is 49 bytes: a 13-byte header followed by a
\nunion whose IMU arm is 36 bytes. For an IMU report joycon_parse_report()
\n-> joycon_parse_imu_report() walks that union (struct offsets 13..48),
\nso a report of exactly 12 bytes with data[0] == JC_INPUT_IMU_DATA passes
\nthe guard yet is read up to 37 bytes past its declared length. The
\nover-read bytes are decoded into accelerometer\/gyroscope values and
\nforwarded to userspace through the “(IMU)” input device, leaking
\ndriver-internal memory. data[0] and size are fully controlled by a
\nmalicious or spoofed Joy-Con\/Pro Controller.<\/p>\n
Receive buffers are sized to the maximum report length, so this is an
\nover-read within the allocation rather than a slab OOB, but the decoded
\nbytes still reach userspace.<\/p>\n
The sibling subcmd path in joycon_ctlr_handle_event() already bounds the
\nsame cast correctly:<\/p>\n
\tif (size < sizeof(struct joycon_input_report) ||
\n\t data[0] != JC_INPUT_SUBCMD_REPLY)
\n\t\tbreak;<\/p>\n
Use the same sizeof(struct joycon_input_report) bound here.<\/p>\n","protected":false},"excerpt":{"rendered":"
In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
HID: nintendo: fix out-of-bounds read in joycon_ctlr_read_handler()<\/p>\n
joycon_ctlr_read_handler() casts an incoming HID input report <\/p>\n","protected":false},"template":"","meta":[],"categories":[],"tags":[],"class_list":["post-17276","vuln","type-vuln","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln\/17276","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln"}],"about":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/types\/vuln"}],"wp:attachment":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=17276"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=17276"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=17276"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}