https://bugzilla.wikimedia.org/show_bug.cgi?id=37225

--- Comment #6 from Aaron Schulz <aschulz4...@gmail.com> 2012-06-05 21:33:11 
UTC ---
I see the line:
    $this->loadPageData( 'fromdbmaster' );
...before the text is loaded in WikiPage::doEdit(). It will the page_latest
field from the master to get the revision/text. That text is compared to the
incoming text to see if anything changed. In fact, wrapped outside of this,
EditPage has a loadPageData( 'forupdate' ) call in internalAttemptSave(). It
doesn't seem like it should be easy for this to happen.

There is a "$dbw->begin( __METHOD__ );" right after $change is checked in
doEdit(). This will release the FOR UPDATE lock. In theory, another process can
save an edit after that and cause a race.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to