diff options
-rw-r--r-- | lib/dma.c | 4 | ||||
-rw-r--r-- | lib/dma.h | 3 |
2 files changed, 6 insertions, 1 deletions
@@ -116,6 +116,10 @@ dma_controller_remove_region(lm_ctx_t *lm_ctx, dma_controller_t *dma, if (region->dma_addr == dma_addr && region->size == size) { _dma_controller_do_remove_region(lm_ctx, region); if (dma->nregions > 1) + /* + * FIXME valgrind complains with 'Source and destination overlap in memcpy', + * check whether memmove eliminates this warning. + */ memcpy(region, &dma->regions[dma->nregions - 1], sizeof *region); dma->nregions--; @@ -139,7 +139,8 @@ dma_addr_to_sg(const dma_controller_t *dma, // Fast path: single region. if (likely(max_sg > 0 && len > 0 && - dma_addr >= region->dma_addr && dma_addr + len <= region_end)) { + dma_addr >= region->dma_addr && dma_addr + len <= region_end && + region_hint < dma->nregions)) { sg->region = region_hint; sg->offset = dma_addr - region->dma_addr; sg->length = len; |