Commit 6d604a10 authored by marko's avatar marko

branches/zip: buf_page_get_gen(): Check the return status of

buf_zip_decompress() and return NULL on decompression failure.
parent c94791e1
...@@ -1712,7 +1712,6 @@ buf_page_get_gen( ...@@ -1712,7 +1712,6 @@ buf_page_get_gen(
buf_block_t* block; buf_block_t* block;
ibool accessed; ibool accessed;
ulint fix_type; ulint fix_type;
ibool success;
ibool must_read; ibool must_read;
ut_ad(mtr); ut_ad(mtr);
...@@ -1788,6 +1787,7 @@ buf_page_get_gen( ...@@ -1788,6 +1787,7 @@ buf_page_get_gen(
switch (buf_block_get_state(block)) { switch (buf_block_get_state(block)) {
buf_page_t* bpage; buf_page_t* bpage;
ibool success;
case BUF_BLOCK_FILE_PAGE: case BUF_BLOCK_FILE_PAGE:
break; break;
...@@ -1904,9 +1904,12 @@ buf_page_get_gen( ...@@ -1904,9 +1904,12 @@ buf_page_get_gen(
/* Decompress the page and apply buffered operations /* Decompress the page and apply buffered operations
while not holding buf_pool->mutex or block->mutex. */ while not holding buf_pool->mutex or block->mutex. */
buf_zip_decompress(block, srv_use_checksums); success = buf_zip_decompress(block, srv_use_checksums);
ibuf_merge_or_delete_for_page(block, space, offset,
zip_size, TRUE); if (UNIV_LIKELY(success)) {
ibuf_merge_or_delete_for_page(block, space, offset,
zip_size, TRUE);
}
/* Unfix and unlatch the block. */ /* Unfix and unlatch the block. */
mutex_enter(&buf_pool->mutex); mutex_enter(&buf_pool->mutex);
...@@ -1917,6 +1920,12 @@ buf_page_get_gen( ...@@ -1917,6 +1920,12 @@ buf_page_get_gen(
mutex_exit(&block->mutex); mutex_exit(&block->mutex);
rw_lock_x_unlock(&block->lock); rw_lock_x_unlock(&block->lock);
if (UNIV_UNLIKELY(!success)) {
mutex_exit(&buf_pool->mutex);
return(NULL);
}
break; break;
case BUF_BLOCK_ZIP_FREE: case BUF_BLOCK_ZIP_FREE:
...@@ -1957,6 +1966,8 @@ buf_page_get_gen( ...@@ -1957,6 +1966,8 @@ buf_page_get_gen(
#endif /* UNIV_DEBUG || UNIV_BUF_DEBUG */ #endif /* UNIV_DEBUG || UNIV_BUF_DEBUG */
if (mode == BUF_GET_NOWAIT) { if (mode == BUF_GET_NOWAIT) {
ibool success;
if (rw_latch == RW_S_LATCH) { if (rw_latch == RW_S_LATCH) {
success = rw_lock_s_lock_func_nowait(&(block->lock), success = rw_lock_s_lock_func_nowait(&(block->lock),
file, line); file, line);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment