Commit 15d8cc6e authored by marko's avatar marko

branches/zip: row_vers_impl_x_locked_off_kernel(): In follow-up to r2119,

assert ut_a(entry) instead of playing it safe.
parent a3190b1f
...@@ -171,10 +171,8 @@ row_vers_impl_x_locked_off_kernel( ...@@ -171,10 +171,8 @@ row_vers_impl_x_locked_off_kernel(
in place of a deleted record, and the BLOB in place of a deleted record, and the BLOB
pointers of the new record were not pointers of the new record were not
initialized yet. But in that case, initialized yet. But in that case,
prev_version should be NULL. prev_version should be NULL. */
ut_a(entry);
We will play it safe and avoid dereferencing
entry when it is NULL, later in this function. */
} }
mutex_enter(&kernel_mutex); mutex_enter(&kernel_mutex);
...@@ -209,7 +207,7 @@ row_vers_impl_x_locked_off_kernel( ...@@ -209,7 +207,7 @@ row_vers_impl_x_locked_off_kernel(
/* We check if entry and rec are identified in the alphabetical /* We check if entry and rec are identified in the alphabetical
ordering */ ordering */
if (entry && 0 == cmp_dtuple_rec(entry, rec, offsets)) { if (0 == cmp_dtuple_rec(entry, rec, offsets)) {
/* The delete marks of rec and prev_version should be /* The delete marks of rec and prev_version should be
equal for rec to be in the state required by equal for rec to be in the state required by
prev_version */ prev_version */
......
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