Skip to content

Commit de8010f

Browse files
committed
Bug#30437378 : ASSERTION N_EXT == DTUPLE->GET_N_EXT() AT
REC_GET_CONVERTED_SIZE() IN REM0REC.IC Issue : In function row_upd_clust_rec_by_insert, n_ext value being passed to lower level functions wasn't correct. Expected value is total number of ext fields whereas value being passed was the number of fields part of index. Fix : Pass the correct value of total number of ext fields. Reviewed by : Annamalai Gurusami <[email protected]>
1 parent 67891b7 commit de8010f

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

storage/innobase/row/row0mysql.cc

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2326,8 +2326,7 @@ row_update_for_mysql_using_cursor(
23262326
if (!dict_index_is_auto_gen_clust(index)) {
23272327
err = row_ins_clust_index_entry(
23282328
index, entry, thr,
2329-
node->upd_ext
2330-
? node->upd_ext->n_ext : 0,
2329+
entry->get_n_ext(),
23312330
true);
23322331
}
23332332
} else {
@@ -2353,7 +2352,7 @@ row_update_for_mysql_using_cursor(
23532352

23542353
err = row_ins_clust_index_entry(
23552354
index, entry, thr,
2356-
node->upd_ext ? node->upd_ext->n_ext : 0,
2355+
entry->get_n_ext(),
23572356
false);
23582357
/* Commit the open mtr as we are processing UPDATE. */
23592358
if (index->last_ins_cur) {

storage/innobase/row/row0upd.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/*****************************************************************************
22
3-
Copyright (c) 1996, 2018, Oracle and/or its affiliates. All Rights Reserved.
3+
Copyright (c) 1996, 2019, Oracle and/or its affiliates. All Rights Reserved.
44
55
This program is free software; you can redistribute it and/or modify
66
it under the terms of the GNU General Public License, version 2.0,
@@ -2589,7 +2589,7 @@ row_upd_clust_rec_by_insert(
25892589

25902590
err = row_ins_clust_index_entry(
25912591
index, entry, thr,
2592-
node->upd_ext ? node->upd_ext->n_ext : 0, false);
2592+
entry->get_n_ext(), false);
25932593
node->state = UPD_NODE_INSERT_CLUSTERED;
25942594

25952595
mem_heap_free(heap);

0 commit comments

Comments
 (0)