diff --git a/.pick_status.json b/.pick_status.json index e54e9eace3c..958a62b288b 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -1164,7 +1164,7 @@ "description": "panvk: Fix IUB decode", "nominated": true, "nomination_type": 2, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "db4bcd48d7a6a775aad608686b865ef584a31861", "notes": null diff --git a/src/panfrost/genxml/decode.c b/src/panfrost/genxml/decode.c index 7003cb7ba39..88928186f1f 100644 --- a/src/panfrost/genxml/decode.c +++ b/src/panfrost/genxml/decode.c @@ -566,7 +566,7 @@ GENX(pandecode_shader)(struct pandecode_context *ctx, uint64_t addr, static unsigned pandecode_buffer(struct pandecode_context *ctx, const struct mali_buffer_packed *cl, uint64_t addr, - uint64_t entry_size) + uint64_t end_of_entry_addr) { pan_unpack(cl, BUFFER, buffer) ; @@ -574,7 +574,7 @@ pandecode_buffer(struct pandecode_context *ctx, /* If the address is the following descriptor and is within the resource * entry, this descriptor is an IUB. */ - if (buffer.address == (addr + 0x20) && buffer.address < addr + entry_size) { + if (buffer.address == (addr + 0x20) && buffer.address < end_of_entry_addr) { assert((buffer.size % 0x20) == 0); const uint8_t *cl_bytes = (uint8_t *)cl; @@ -623,7 +623,7 @@ pandecode_resources(struct pandecode_context *ctx, uint64_t addr, unsigned size) break; case MALI_DESCRIPTOR_TYPE_BUFFER: i += pandecode_buffer(ctx, (const struct mali_buffer_packed *)&cl[i], - addr + i, size); + addr + i, addr + size); break; default: fprintf(ctx->dump_stream, "Unknown descriptor type %X\n", header.type);