Commit 880d0295 authored by unknown's avatar unknown
Browse files

row0sel.c: 

  Do not test the value of err if the lock operation was skipped because innodb_logs_unsafe_for_binlog was TRUE; though this did not cause any bugs visible to the user, because err is inited to DB_SUCCESS at the start of the function row_search_for_mysql()


innobase/row/row0sel.c:
  Do not test the value of err if the lock operation was skipped because innodb_logs_unsafe_for_binlog was TRUE; though this did not cause any bugs visible to the user, because err is inited to DB_SUCCESS at the start of the function row_search_for_mysql()
parent dec90f5b

innobase/row/row0sel.c

+12 −12
Original line number Diff line number Diff line
@@ -3305,13 +3305,13 @@ row_search_for_mysql( 
				err = sel_set_rec_lock(rec, index,

						prebuilt->select_lock_type,

						LOCK_ORDINARY, thr);

			}



				if (err != DB_SUCCESS) {



					goto lock_wait_or_error;

				}

			}



		}

		/* A page supremum record cannot be in the result set: skip

		it now that we have placed a possible lock on it */
@@ -3413,14 +3413,14 @@ row_search_for_mysql( 
					err = sel_set_rec_lock(rec, index,

						prebuilt->select_lock_type,

						LOCK_GAP, thr);

				}



					if (err != DB_SUCCESS) {



						goto lock_wait_or_error;

					}

				}



			}



			btr_pcur_store_position(pcur, &mtr);



			ret = DB_RECORD_NOT_FOUND;
@@ -3446,14 +3446,14 @@ row_search_for_mysql( 
					err = sel_set_rec_lock(rec, index,

						prebuilt->select_lock_type,

						LOCK_GAP, thr);

				}



					if (err != DB_SUCCESS) {



						goto lock_wait_or_error;

					}

				}



			}



			btr_pcur_store_position(pcur, &mtr);



			ret = DB_RECORD_NOT_FOUND;