fix incorrect MTP xact_len calculation

RigoLigoRLC · RigoLigoRLC · commit bf69c49c29a5 · 2025-10-08T07:22:34.000+08:00
diff --git a/src/class/mtp/mtp_device.c b/src/class/mtp/mtp_device.c
@@ -215,7 +215,7 @@ static bool mtpd_data_xfer(mtp_container_info_t* p_container, uint8_t ep_addr) {
     TU_ASSERT(p_mtp->phase == MTP_PHASE_DATA);
   }
 
-  const uint16_t xact_len = tu_min16((uint16_t) (p_mtp->total_len - p_mtp->xferred_len), CFG_TUD_MTP_EP_BUFSIZE);
+  const uint16_t xact_len = (uint16_t) tu_min32(p_mtp->total_len - p_mtp->xferred_len, CFG_TUD_MTP_EP_BUFSIZE);
   if (xact_len) {
     // already transferred all bytes in header's length. Application make an unnecessary extra call
     TU_VERIFY(usbd_edpt_claim(p_mtp->rhport, ep_addr));

Original file line number	Diff line number	Diff line change
`@@ -215,7 +215,7 @@ static bool mtpd_data_xfer(mtp_container_info_t* p_container, uint8_t ep_addr) {`
`215`	`215`	`TU_ASSERT(p_mtp->phase == MTP_PHASE_DATA);`
`216`	`216`	`}`
`217`	`217`
`218`		`- const uint16_t xact_len = tu_min16((uint16_t) (p_mtp->total_len - p_mtp->xferred_len), CFG_TUD_MTP_EP_BUFSIZE);`
	`218`	`+ const uint16_t xact_len = (uint16_t) tu_min32(p_mtp->total_len - p_mtp->xferred_len, CFG_TUD_MTP_EP_BUFSIZE);`
`219`	`219`	`if (xact_len) {`
`220`	`220`	`// already transferred all bytes in header's length. Application make an unnecessary extra call`
`221`	`221`	`TU_VERIFY(usbd_edpt_claim(p_mtp->rhport, ep_addr));`