Claude review: media: pluto2: fix potential buffer overflow in pluto_dma_end()

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Assessment: Good fix, minor nit**

The bug is real. `TS_DMA_BYTES` is `188 * 8 = 1504`, and `dma_buf` is declared as `u8 dma_buf[TS_DMA_BYTES]` (line 111). The original loop:

```c
while (pluto->dma_buf[i] == 0x47)
    i += 188;
```

increments `i` by 188 each iteration. If every 188th byte happens to be `0x47`, `i` would reach `188 * 8 = 1504 = TS_DMA_BYTES`, and the next iteration would read `pluto->dma_buf[1504]` — one byte past the end of the buffer. The fix:

```c
while (i < TS_DMA_BYTES && pluto->dma_buf[i] == 0x47)
    i += 188;
```

correctly bounds-checks before accessing the buffer. Short-circuit evaluation ensures `dma_buf[i]` is never accessed when `i >= TS_DMA_BYTES`.

**Minor observation:** In practice, the maximum valid `i` after the loop would be `TS_DMA_BYTES` (1504), and `nbpackets = 1504 / 188 = 8 = TS_DMA_PACKETS`, which is the correct maximum. So the fix is semantically clean — it doesn't change behavior for valid data, only prevents the OOB read.

**Verdict:** Straightforward and correct one-line fix. The commit message, Fixes tag, and Signed-off-by are all proper.

Reviewed-by assessment: **Accept**.

---
Generated by Claude Code Patch Reviewer