Avoid passing null to strcasecmp(), for PHP 8.1

Bug: T307282 Change-Id: I523928b3f5e0e02e23c45e7023d9d2701d986e5c
wikimedia · Jun 11, 2022 · 2a91787 · 2a91787
1 parent 2efa42f
commit 2a91787
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/includes/session/CookieSessionProvider.php b/includes/session/CookieSessionProvider.php
@@ -98,8 +98,8 @@ protected function postInitSetup() {
 				?: $this->getConfig()->get( MainConfigNames::CookiePrefix ) . '_session',
 		];
 
-		$this->useCrossSiteCookies =
-			strcasecmp( $this->getConfig()->get( MainConfigNames::CookieSameSite ), 'none' ) === 0;
+		$sameSite = $this->getConfig()->get( MainConfigNames::CookieSameSite );
+		$this->useCrossSiteCookies = $sameSite !== null && strcasecmp( $sameSite, 'none' ) === 0;
 
 		// @codeCoverageIgnoreStart
 		$this->cookieOptions += [
@@ -110,7 +110,7 @@ protected function postInitSetup() {
 			'secure' => $this->getConfig()->get( MainConfigNames::CookieSecure )
 				|| $this->getConfig()->get( MainConfigNames::ForceHTTPS ),
 			'httpOnly' => $this->getConfig()->get( MainConfigNames::CookieHttpOnly ),
-			'sameSite' => $this->getConfig()->get( MainConfigNames::CookieSameSite ),
+			'sameSite' => $sameSite,
 		];
 	}