mirror of
https://github.com/MariaDB/server.git
synced 2025-01-15 19:42:28 +01:00
97bd763544
Accessing well defined MERGE table may return an error
stating that the merge table is incorrectly defined. This
happens if MERGE child tables were accessed before and we
failed to open another incorrectly defined MERGE table in
this connection.
myrg_open() internally used my_errno as a variable for determining
failure, and thus could be tricked into a wrong decision by other
uses of my_errno.
With this fix we use function local boolean flag instead of my_errno
to determine failure.
myisammrg/myrg_open.c:
There are two requirement for accessing/setting my_errno variable,
which were not followed by myrg_open():
- it must be checked immediately after a function returned an error. There
must be no calls to other functions that may change it's value between.
- my_errno value must be set right before a function is going to return an
error. There must be no calls to other functions that may change it's
value between (that's why we have these tricks with save_errno at the
bottom of myrg_open()).
myrg_open() internally used my_errno as a variable for determining
failure, and thus could be tricked into a wrong decision by other
uses of my_errno.
mysql-test/r/merge.result:
A test case for BUG#32047.
mysql-test/t/merge.test:
A test case for BUG#32047.
|
||
|---|---|---|
| .. | ||
| .cvsignore | ||
| CMakeLists.txt | ||
| make-ccc | ||
| Makefile.am | ||
| myrg_close.c | ||
| myrg_create.c | ||
| myrg_def.h | ||
| myrg_delete.c | ||
| myrg_extra.c | ||
| myrg_info.c | ||
| myrg_locking.c | ||
| myrg_open.c | ||
| myrg_panic.c | ||
| myrg_queue.c | ||
| myrg_range.c | ||
| myrg_rfirst.c | ||
| myrg_rkey.c | ||
| myrg_rlast.c | ||
| myrg_rnext.c | ||
| myrg_rnext_same.c | ||
| myrg_rprev.c | ||
| myrg_rrnd.c | ||
| myrg_rsame.c | ||
| myrg_static.c | ||
| myrg_update.c | ||
| myrg_write.c | ||