Merge branch 'krille/fix-missing-null-check' into 'main'

fix: Missing null check which F*cks me up!

See merge request famedly/famedlysdk!563
This commit is contained in:
Nicolas Werner 2020-12-14 13:24:39 +00:00
commit 03eba89297
2 changed files with 3 additions and 2 deletions

View File

@ -17,7 +17,7 @@ coverage:
- sed -i s/#//g pubspec.yaml
- flutter doctor
- apt update
- apt install -y chromium lcov libolm3 sqlite3 libsqlite3-dev
- apt install -y chromium-bsu lcov libolm3 sqlite3 libsqlite3-dev
- ln -s /usr/lib/dart/bin/pub /usr/bin/
- useradd -m test
- chown -R 'test:' '.'
@ -62,7 +62,7 @@ e2ee_test:
- curl https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add -
- curl https://storage.googleapis.com/download.dartlang.org/linux/debian/dart_stable.list > /etc/apt/sources.list.d/dart_stable.list
- apt update
- apt install -y dart chromium lcov libolm3 sqlite3 libsqlite3-dev
- apt install -y dart chromium-bsu lcov libolm3 sqlite3 libsqlite3-dev
- ln -s /usr/lib/dart/bin/pub /usr/bin/
- useradd -m test
- chown -R 'test:' '.'

View File

@ -184,6 +184,7 @@ abstract class SignableKey extends MatrixSignableKey {
return false; // prevent recursion & validate hasValidSignatureChain
}
visited.add(setKey);
if (signatures == null) return false;
for (final signatureEntries in signatures.entries) {
final otherUserId = signatureEntries.key;
if (!(signatureEntries.value is Map) ||