lsp: several bug fixes with registration

LMMS · May 15, 2021 · 02aa856 · 02aa856
1 parent 6050a66
commit 02aa856
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/public/lsp/register.php b/public/lsp/register.php
@@ -45,7 +45,7 @@ function try_add_user($login , $pass, $pass2, $realname, $session, $is_admin, $a
 function check_antispam($antispam) {
 	if (isset($antispam)) {
 		for ($i = 0; $i < 25; $i++) {
-			$md5 = md5(session_id() + $i);
+			$md5 = md5(intval(session_id()) + $i);
 			if (strpos($antispam, substr($md5, strlen("$md5") - 4, strlen("$md5"))) !== false) {
 				return true;
 			}

diff --git a/public/lsp/utils.php b/public/lsp/utils.php
@@ -290,6 +290,9 @@ function read_project($file_id) {
 		case '.mmpz' :
 			// Open binary file for reading
 			$handle = fopen($DATA_DIR . $file_id, "rb");
+			if (!$handle) {
+				return null;
+			}
 			// Skip the first 4 bytes for compressed mmpz files
 			fseek($handle, 4);
 			$data = fread($handle, filesize($DATA_DIR . $file_id) - 4);

diff --git a/public/utils.php b/public/utils.php
@@ -1,12 +1,5 @@
 <?php
 
-function str_contains($haystack, $needle, $ignorecase = FALSE) {
-	if ($ignorecase) {
-		return strpos(strtolower($haystack), strtolower($needle)) !== FALSE;
-	}
-	return strpos($haystack, $needle) !== FALSE;
-}
-
 function str_startswith($haystack, $needle, $ignorecase = FALSE) {
 	if ($ignorecase) {
 		return $needle === "" || strpos(strtolower($haystack), strtolower($needle)) === 0;