Rollbacks improperly reported under race condition
Closed, ResolvedPublic


Author: Winbots

Fixes bug.

Rollbacks can be improperly reported under certain race conditions. Mainly because the doEdit call's status is never checked before notifying of a successful rollback.

The diff should fix this.


Version: 1.17.x
Severity: enhancement

Attached: racecondition.diff


bzimport set Reference to bz24866.
bzimport added a subscriber: Unknown Object (MLST).
bzimport created this task.Aug 19 2010, 5:23 AM

Using the message "Cannot revert edit; last contributor is only author of this page." is bad. A new message should be added.

Reedy added a comment.Aug 19 2010, 9:27 AM

You should really have it with braces as per the coding style guidelines

if( !$status->isOK() ) {
return array( array( 'cantrollback' ) );

Bryan.TongMinh wrote:

Marking as reviewed per comment #2.

Check for $status->isOk was added with gerrit 21662

Add Comment