Type check the APC value in LoadBalancer::doWait()
authorAaron Schulz <aschulz@wikimedia.org>
Fri, 10 Feb 2017 20:00:44 +0000 (12:00 -0800)
committerAaron Schulz <aschulz@wikimedia.org>
Fri, 10 Feb 2017 20:00:44 +0000 (12:00 -0800)
Change-Id: If00cd5fd2ace828ce03824ec7b5735de7d950dec

includes/libs/rdbms/loadbalancer/LoadBalancer.php

index 532a8ed..900a79c 100644 (file)
@@ -491,7 +491,10 @@ class LoadBalancer implements ILoadBalancer {
                $key = $this->srvCache->makeGlobalKey( __CLASS__, 'last-known-pos', $server );
                /** @var DBMasterPos $knownReachedPos */
                $knownReachedPos = $this->srvCache->get( $key );
-               if ( $knownReachedPos && $knownReachedPos->hasReached( $this->mWaitForPos ) ) {
+               if (
+                       $knownReachedPos instanceof DBMasterPos &&
+                       $knownReachedPos->hasReached( $this->mWaitForPos )
+               ) {
                        $this->replLogger->debug( __METHOD__ .
                                ": replica DB $server known to be caught up (pos >= $knownReachedPos)." );
                        return true;