In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
scsi: target: Bound PR-OUT TransportID parsing to the received buffer<\/p>\n
core_scsi3_decode_spec_i_port() and core_scsi3_emulate_register_and_move()
\nhand the raw PERSISTENT RESERVE OUT parameter buffer to
\ntarget_parse_pr_out_transport_id() without telling it how many bytes are
\nvalid. For an iSCSI TransportID (FORMAT CODE 01b),
\niscsi_parse_pr_out_transport_id() locates the “,i,0x” ISID separator with
\nan unbounded strstr() (and on the error path prints the name with a further
\nunbounded “%s”). An initiator can submit a TransportID whose iSCSI name
\ncontains neither a “,i,0x” substring nor a NUL terminator, filling the
\nparameter list to its end, so the scan runs off the end of the buffer.<\/p>\n
When the parameter list spans more than one page the buffer is a multi-page
\nvmap (transport_kmap_data_sg()), so the over-read walks into the trailing
\nvmalloc guard page and oopses (KASAN: vmalloc-out-of-bounds in strstr). It
\nis reachable by any fabric that delivers a PR OUT to a device exported
\nthrough an iSCSI TPG, including a guest via vhost-scsi.<\/p>\n
Pass the number of received bytes down to the parser and validate the iSCSI
\nTransportID’s own self-described length (ADDITIONAL LENGTH + 4) once, up
\nfront: reject it if it is below the spc4r17 minimum or larger than the
\nreceived buffer, then bound the separator search, the ISID walk and the
\nname copy by that length. This is the length check the callers already
\nperform after the parse (core_scsi3_decode_spec_i_port() compares tid_len
\nagainst tpdl, core_scsi3_emulate_register_and_move() validates it against
\ndata_length), moved ahead of the scan. Also drop the unbounded “%s” of the
\nunterminated name.<\/p>\n
Add per-format explicit name-length checks before copying into i_str,
\nrather than silently truncating with min_t: for FORMAT CODE 00b reject if
\nthe descriptor body (tid_len – 4 bytes) cannot fit in
\ni_str[TRANSPORT_IQN_LEN]; for FORMAT CODE 01b reject if the name portion
\n(from &buf[4] up to the separator) cannot fit. Both checks make the bounds
\nintent explicit at each format branch.<\/p>\n
While here, also reject a FORMAT CODE 01b TransportID whose “,i,0x”
\nseparator sits at the very end of the descriptor: that leaves an empty ISID
\nand points the returned port nexus pointer at buf + tid_len, one past the
\ndescriptor, which the registration code (__core_scsi3_locate_pr_reg(),
\n__core_scsi3_alloc_registration()) then dereferences as the ISID string —
\nthe same over-read of the parameter buffer for a malformed descriptor.<\/p>\n","protected":false},"excerpt":{"rendered":"
In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
scsi: target: Bound PR-OUT TransportID parsing to the received buffer<\/p>\n
core_scsi3_decode_spec_i_port() and core_scsi3_emulate_regis<\/p>\n","protected":false},"template":"","meta":[],"categories":[],"tags":[],"class_list":["post-6763","vuln","type-vuln","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln\/6763","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=6763"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=6763"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=6763"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}