Skip to content

Commit 02bcdcc

Browse files
committed
Avoid updating context->bstarts, as it is not needed
1 parent 5a2b0ed commit 02bcdcc

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

blosc/blosc2.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3059,7 +3059,7 @@ int _blosc_getitem(blosc2_context* context, blosc_header* header, const void* sr
30593059
return BLOSC2_ERROR_WRITE_BUFFER;
30603060
}
30613061

3062-
context->bstarts = (int32_t*)(_src + context->header_overhead);
3062+
int32_t* bstarts = (int32_t*)(_src + context->header_overhead);
30633063

30643064
/* Check region boundaries */
30653065
if ((start < 0) || (start * header->typesize > header->nbytes)) {
@@ -3074,7 +3074,7 @@ int _blosc_getitem(blosc2_context* context, blosc_header* header, const void* sr
30743074

30753075
int chunk_memcpy = header->flags & 0x1;
30763076
if (!context->special_type && !chunk_memcpy &&
3077-
((uint8_t *)(_src + srcsize) < (uint8_t *)(context->bstarts + context->nblocks))) {
3077+
((uint8_t *)(_src + srcsize) < (uint8_t *)(bstarts + context->nblocks))) {
30783078
BLOSC_TRACE_ERROR("`bstarts` out of bounds.");
30793079
return BLOSC2_ERROR_READ_BUFFER;
30803080
}
@@ -3178,7 +3178,7 @@ int _blosc_getitem(blosc2_context* context, blosc_header* header, const void* sr
31783178

31793179
// If memcpyed we don't have a bstarts section (because it is not needed)
31803180
int32_t src_offset = memcpyed ?
3181-
context->header_overhead + j * header->blocksize : sw32_(context->bstarts + j);
3181+
context->header_overhead + j * header->blocksize : sw32_(bstarts + j);
31823182

31833183
int32_t cbytes = blosc_d(context->serial_context, bsize, leftoverblock, memcpyed,
31843184
src, srcsize, src_offset, j,

0 commit comments

Comments
 (0)