Added user Id sanity check to setUsernameBitfields() for bug 34755.

Change-Id: I44288d199d86045151f8184f7abaa3df886d9d99

diff --git a/includes/revisiondelete/RevisionDeleteUser.php b/includes/revisiondelete/RevisionDeleteUser.php
index fe424ef..0e965c8 100644 (file)
--- a/includes/revisiondelete/RevisionDeleteUser.php
+++ b/includes/revisiondelete/RevisionDeleteUser.php
@@ -34,10 +34,10 @@ class RevisionDeleteUser {
         * @return bool
         */
        private static function setUsernameBitfields( $name, $userId, $op, $dbw ) {
-               if( $op !== '|' && $op !== '&' ){
+               if ( !$userId || ( $op !== '|' && $op !== '&' ) ) {
                        return false; // sanity check
                }
-               if( !$dbw instanceof DatabaseBase ){
+               if ( !$dbw instanceof DatabaseBase ) {
                        $dbw = wfGetDB( DB_MASTER );
                }