diff --git a/app/src/main/java/io/legado/app/data/entities/BaseSource.kt b/app/src/main/java/io/legado/app/data/entities/BaseSource.kt
index 79c8cdae8ced..275835fae88a 100644
--- a/app/src/main/java/io/legado/app/data/entities/BaseSource.kt
+++ b/app/src/main/java/io/legado/app/data/entities/BaseSource.kt
@@ -67,7 +67,7 @@ interface BaseSource : JsExtensions {
     fun loginUi(): List<RowUi>? {
         return GSON.fromJsonArray<RowUi>(loginUi).onFailure {
             it.printOnDebug()
-        }.getOrNull()
+        }.getOrNull()?.filterNotNull() // filter null, see https://github.com/gedoor/legado/discussions/4650
     }
 
     fun getLoginJs(): String? {
@@ -253,4 +253,4 @@ interface BaseSource : JsExtensions {
     fun getShareScope(): Scriptable? {
         return SharedJsScope.getScope(jsLib)
     }
-}
\ No newline at end of file
+}